----------------------------------------------------------- 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
