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

(Updated Nov. 19, 2015, 1:21 a.m.)


Review request for samza, Yan Fang, Jake Maes, Jagadish Venkatraman, Xinyu Liu, 
and Yi Pan (Data Infrastructure).


Changes
-------

I was able to add only 1 unit test for now.


Bugs: SAMZA-817
    https://issues.apache.org/jira/browse/SAMZA-817


Repository: samza


Description
-------

Adding ratio of successful locality matched request as a gauge


Diffs (updated)
-----

  samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala 
3787b85e5cc738d16c0d1eaea4de3345a3a9106c 
  
samza-core/src/main/scala/org/apache/samza/container/SamzaContainerMetrics.scala
 127f3a141f25930281ef3c64b314a7b5b79dfade 
  samza-core/src/test/scala/org/apache/samza/container/TestSamzaContainer.scala 
d91b1dae3b2df5dd2fe8714e1b3b2eb4bbb74293 
  
samza-yarn/src/main/java/org/apache/samza/job/yarn/AbstractContainerAllocator.java
 6edd47717ff10fbf10ce52b401b8ce093c59930e 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerUtil.java 
55dbfeac44834f9899b473318fb7dcefe7a59bc3 
  
samza-yarn/src/main/java/org/apache/samza/job/yarn/HostAwareContainerAllocator.java
 ad1587d6e00b7145bbe752047fcd9046c7b1b73e 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaAppState.java 
3df927e5dfd056b00a77f86aaba49585264115ae 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterMetrics.scala 
ae5a674f1c29e1d05be17a037dbe38b033a45145 

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


Testing
-------

./gradlew clean build


Thanks,

Navina Ramesh

Reply via email to