> On June 8, 2016, 7:53 p.m., Yi Pan (Data Infrastructure) wrote: > > samza-yarn/src/main/resources/scalate/WEB-INF/views/index.scaml, line 18 > > <https://reviews.apache.org/r/48393/diff/1/?file=1410828#file1410828line18> > > > > A quick question: is YarnAppState going to be a super set of > > SamzaAppState? Or reverse? I thought that w/ the refactoring, we are > > shooting for a reverse logic: SamzaAppState would be the super set that > > includes YARN-specific states?
Or, if the SamzaAppState is the class that should make sense to general Samza applications w/ or w/o YARN. Can we define YarnAppState as an interface and let a derived SamzaYarnAppState extends SamzaAppState implements YarnAppState? - Yi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48393/#review136675 ----------------------------------------------------------- On June 8, 2016, 4:03 a.m., Jagadish Venkatraman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48393/ > ----------------------------------------------------------- > > (Updated June 8, 2016, 4:03 a.m.) > > > Review request for samza and Yi Pan (Data Infrastructure). > > > Repository: samza > > > Description > ------- > > Integrate Kerberos with the refactored JC patch as in RB:47283. > > > Diffs > ----- > > > samza-core/src/main/java/org/apache/samza/clustermanager/AbstractContainerAllocator.java > 097a476bee75bb2b5a56ae85317fa16523baebd3 > > samza-core/src/main/java/org/apache/samza/clustermanager/ContainerProcessManager.java > f2db34c24b2b2ef05705a2fbf00d4a211d6f4c58 > > samza-core/src/main/java/org/apache/samza/clustermanager/SamzaApplicationState.java > ca277b3636aa4e98e639699e657de6aced613c0f > > samza-core/src/main/scala/org/apache/samza/metrics/ContainerProcessManagerMetrics.scala > 86c2440ad1d253fd71030916915984c1d08614fb > > samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerAllocator.java > f1475709c2db364110b00cb04cf55c6ecb44fd04 > > samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerProcessManager.java > 4fd1018f93dcfbccf70595c4bcdec956d2b59295 > > samza-core/src/test/java/org/apache/samza/clustermanager/TestHostAwareContainerAllocator.java > 57fef126de535067a68ce6eb8eeddf3334d1fe5d > samza-shell/src/main/bash/run-am.sh > ca938cc910b8b59e2cdab68df9f9ee16b425fb74 > > samza-yarn/src/main/java/org/apache/samza/job/yarn/AbstractContainerAllocator.java > b4789e62beb1120f11a8101664b10c34ae930e58 > samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerAllocator.java > 24ac41067e34cf5a445bb036db9ea324eaafa7df > samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerFailure.java > 1d15651096e52a2e323a8e2f658fad3ea8e9c709 > > samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerRequestState.java > 57ce35099973b7fc3414c450e3246cb9f204289b > samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerUtil.java > e8976bce934a324c48475bcd64d392119cc44b40 > > samza-yarn/src/main/java/org/apache/samza/job/yarn/HostAwareContainerAllocator.java > 1d101fa80e6367d54f06455c242222195e4c0091 > samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaAppState.java > c116ed83f8b424bc7353e89c60cf165da542fbec > > samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaContainerRequest.java > 4a04eb6b3b054ee85988e1b26ececc800bbc7861 > samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaTaskManager.java > bc95f31c0dcaaa68d483a6f152b61aba6c543fff > samza-yarn/src/main/java/org/apache/samza/job/yarn/YarnAppState.java > 57092e1b100b0128aed2a1426835d7184cc5bef7 > > samza-yarn/src/main/java/org/apache/samza/job/yarn/YarnClusterResourceManager.java > 7778a380dcd99139012a71c80fb86ede1fef0f71 > samza-yarn/src/main/resources/scalate/WEB-INF/views/index.scaml > 93660c73e507b1a6076e743fcec56a41d22a5f39 > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMaster.scala > 7bd813159745b105cd31c8e19f1352280acf9da5 > > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterLifecycle.scala > 2a5c0d8092aa6f0a7f9b1f5fa56f9f4d4919d579 > > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterMetrics.scala > 054d8b68033535ff0cab7cca84b71455e201a715 > > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterService.scala > 3adf86f83123f6d66fc18ef9feed95b551c8398f > > samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaYarnAppMasterService.scala > f62bec1b305b7301829644b5558eac197ea7457e > samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnJob.scala > 62ddb261f0b8f0f24468875d6d84da05908a3c62 > > samza-yarn/src/main/scala/org/apache/samza/webapp/ApplicationMasterRestServlet.scala > a40ab72a71c4fa82bd87aa03caec8936e609bf68 > > samza-yarn/src/main/scala/org/apache/samza/webapp/ApplicationMasterWebServlet.scala > 605332a02f365248fc9ec3525ed9d3721db71e8d > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerAllocator.java > e21aded3c5e302593ed1cb5675da81aa6e3e743f > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerAllocatorCommon.java > 0bbd48d8720a79f3b453d69ca2772f0db10723be > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerRequestState.java > 402fe784120ac40ed542d9fa60d6a6d7df9c8cda > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestHostAwareContainerAllocator.java > ead7200487f27ed31e30576721ef16a064a28bc7 > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestSamzaContainerRequest.java > ad0f4d3d9db9111a7c0087b6da9c724dcc736726 > > samza-yarn/src/test/java/org/apache/samza/job/yarn/TestSamzaTaskManager.java > d747b818ffdf0c9968afdf098b2a3f14b7b02a56 > > samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerAllocator.java > 329024729eeae07f8e755777b7211772cd98e04e > > samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerRequestState.java > 7c0b50437b06230ab840fdad894516c6d8545b4a > > samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerUtil.java > cf3e1439d005ca03f8c0e84183a26a0ca85abd0a > samza-yarn/src/test/java/org/apache/samza/job/yarn/util/TestUtil.java > d4c9c96144c58e1acc53726265a452f2467cd554 > samza-yarn/src/test/scala/org/apache/samza/job/yarn/MockSystemAdmin.scala > PRE-CREATION > > samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMaster.scala > 3f056c4d39fc8e3a2ed42fbcc73aa6eca1a2287c > > samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterLifecycle.scala > 750f467da12c5eb7ba80b06faadb1bac12feac1b > > samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterService.scala > fc0091f3f3ba0b2e7cd6bbd4e279b71087fb326b > > Diff: https://reviews.apache.org/r/48393/diff/ > > > Testing > ------- > > Manual testing. > > Added Unit tests. > > Manually tested with a sample hello-world job. > > Tested that the UI was showing up correctly and template files were being > populated. f > > > Thanks, > > Jagadish Venkatraman > >
