samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jagadish Venkatraman <jagadish1...@gmail.com>
Subject Re: Review Request 51126: SAMZA 998: Documentation updates for refactored Job Coordinator
Date Fri, 16 Sep 2016 21:45:38 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51126/
-----------------------------------------------------------

(Updated Sept. 16, 2016, 9:45 p.m.)


Review request for samza, Jake Maes, Yi Pan (Data Infrastructure), and Navina Ramesh.


Changes
-------

1.Address Yi's review feedback. 
2.Re-organize some configs
3.Make host affinity config: job.host-affinity.enabled.


Repository: samza


Description
-------

This documents the new config properties that were changed as a result of SAMZA-680. Some
config properties that were specific to Yarn have been moved to the "cluster manager" namespace.


Question:
1.Is there a better prefix name we can come up with for config properties in the new name-space
- instead of "cluster-manager.*" , how about job.*?

Suggestions for better config property names are welcome :-)


Diffs (updated)
-----

  docs/learn/documentation/versioned/jobs/configuration-table.html 54c52981c3055b398ee60af50eeaf2592ed0e64f

  samza-core/src/main/java/org/apache/samza/config/ClusterManagerConfig.java dafd7a7ac3f7c38aeab68de2dabfe09dde082388


Diff: https://reviews.apache.org/r/51126/diff/


Testing
-------

1. Generated html builds clean.
2. Built and tested the Samza website locally - `?  docs git:(doc-updates) ? bundle exec jekyll
serve --watch --baseurl`


Thanks,

Jagadish Venkatraman


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message