Re: Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-10-12 Thread Maxim Logvinenko
ews.apache.org/r/51633/#review152360 --- On Oct. 7, 2016, 12:08 a.m., Maxim Logvinenko wrote: > > --- > This is an automatically generated e-mail. To reply

Re: Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-10-07 Thread Maxim Logvinenko
> Can you please add some detail on testing this feature? > > What was the label setup of the cluster? (for example: Did we use an > > exclusive node label?), How many node labels? How many containers were > > requested for the job? > > Maxim Logvinenko wrote: > We

Re: Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-10-06 Thread Maxim Logvinenko
generated e-mail. To reply, visit: https://reviews.apache.org/r/51633/#review151519 ------- On Oct. 7, 2016, 12:08 a.m., Maxim Logvinenko wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apa

Re: Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-10-06 Thread Maxim Logvinenko
request has resourceName = ANY. - Maxim --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51633/#review151519 --- On Oct. 7, 2016, 12:08 a.m.

Re: Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-10-06 Thread Maxim Logvinenko
n/src/main/java/org/apache/samza/job/yarn/YarnClusterResourceManager.java 96d3d7c samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala 0998c43 Diff: https://reviews.apache.org/r/51633/diff/ Testing --- Thanks, Maxim Logvinenko

Review Request 51633: SAMZA-1013: Add YARN Node label support

2016-09-05 Thread Maxim Logvinenko
samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala 0998c43 Diff: https://reviews.apache.org/r/51633/diff/ Testing --- Thanks, Maxim Logvinenko

Review Request 51634: SAMZA-1014: Add property to set YARN AM cpu cores

2016-09-05 Thread Maxim Logvinenko
ache.org/r/51634/diff/ Testing --- Thanks, Maxim Logvinenko