----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36089/#review91533 -----------------------------------------------------------
samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala (line 634) <https://reviews.apache.org/r/36089/#comment144956> I think a better way, which requires much fewer changes, is to call something like jmxServer.getJmxUrl, jmxServer.jmxTunelingUrl. jmxServer can be a variable of SamzaContainer Object. Then we do not need to change ContainerModel, JobModel, SamzaContext. Because there is no reason that we want to contain jmx information into those three objects. - Yan Fang On July 1, 2015, 2:07 p.m., József Márton Jung wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36089/ > ----------------------------------------------------------- > > (Updated July 1, 2015, 2:07 p.m.) > > > Review request for samza. > > > Repository: samza > > > Description > ------- > > JMX address of application master and the containers is available through AM > UI > > > Diffs > ----- > > checkstyle/import-control.xml 3374f0c > > samza-api/src/main/java/org/apache/samza/container/SamzaContainerContext.java > fd7333b > samza-core/src/main/java/org/apache/samza/container/LocalityManager.java > e661e12 > > samza-core/src/main/java/org/apache/samza/coordinator/stream/CoordinatorStreamMessage.java > 6c1e488 > samza-core/src/main/java/org/apache/samza/job/model/ContainerModel.java > 98a34bc > samza-core/src/main/java/org/apache/samza/job/model/JobModel.java 95a2ce5 > samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala > cbacd18 > samza-core/src/main/scala/org/apache/samza/coordinator/JobCoordinator.scala > 8ee034a > samza-core/src/main/scala/org/apache/samza/metrics/JmxServer.scala f343faf > > samza-core/src/test/scala/org/apache/samza/container/TestSamzaContainer.scala > 9fb1aa9 > samza-core/src/test/scala/org/apache/samza/container/TestTaskInstance.scala > 7caad28 > > samza-test/src/main/scala/org/apache/samza/test/performance/TestKeyValuePerformance.scala > 1ce7d25 > samza-yarn/src/main/resources/scalate/WEB-INF/views/index.scaml cf0d2fc > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMaster.scala > 20aa373 > > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterState.scala > 1445605 > > Diff: https://reviews.apache.org/r/36089/diff/ > > > Testing > ------- > > > Thanks, > > József Márton Jung > >
