You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@samza.apache.org by Jagadish Venkatraman <ja...@gmail.com> on 2016/03/19 01:29:14 UTC

Review Request 45063: SAMZA-903

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

Review request for samza.


Repository: samza


Description
-------

As a part of the refactoring we separate out Samza specific state and Yarn specific state. As a result, the class SamzaAppState is moved from samza-yarn to samza-core.
The variables in the old SamzaAppState are now split into 
i) Refactored SamzaAppState (in samza core)
ii)YarnAppState (in samza yarn)
Hence, we will need to change the references to SamzaAppState in all the UI scala template classes to work with the newer refactoring. This RB creates a scalatev2 directory that will include the refactored templates.

This RB depends on RB:44920 

Review tools does not support a way to upload a diff against some already existing change-list. Hence, this RB is deceptively large.


Diffs
-----

  build.gradle 16facbbf4dff378c561461786ff186bd9e0000ed 
  samza-core/src/main/java/org/apache/samza/clustermanager/AbstractContainerAllocator.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ClusterBasedJobCoordinator.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ContainerAllocator.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ContainerManagerFactory.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ContainerProcessManager.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ContainerRequestState.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/HostAwareContainerAllocator.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/ResourceFailure.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaAppState.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaContainerLaunchException.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaResource.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaResourceRequest.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaResourceStatus.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/clustermanager/SamzaTaskManager.java PRE-CREATION 
  samza-core/src/main/java/org/apache/samza/config/ClusterManagerConfig.java PRE-CREATION 
  samza-core/src/main/scala/org/apache/samza/coordinator/JobModelReader.scala PRE-CREATION 
  samza-core/src/main/scala/org/apache/samza/metrics/SamzaAppMasterMetrics.scala PRE-CREATION 
  samza-shell/src/main/bash/run-am.sh 9545a96953baaff17ad14962e02bc12aadbb1101 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/refactor/YarnAppState.java PRE-CREATION 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/refactor/YarnContainerManager.java PRE-CREATION 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/refactor/YarnContainerManagerFactory.java PRE-CREATION 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/refactor/YarnContainerRunner.java PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/WEB-INF/layouts/default.scaml PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/WEB-INF/views/index.scaml PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/css/bootstrap.min.css PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/css/font-awesome.min.css PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/css/ropa-sans.css PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/FontAwesome.otf PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/RopaSans-Regular-webfont.woff PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/fontawesome-webfont.eot PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/fontawesome-webfont.svg PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/fontawesome-webfont.ttf PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/fontawesome-webfont.woff PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/glyphicons-halflings-regular.eot PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/glyphicons-halflings-regular.svg PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/glyphicons-halflings-regular.ttf PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/fonts/glyphicons-halflings-regular.woff PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/img/asc.gif PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/img/bg.gif PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/img/desc.gif PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/img/samza-icon.png PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/js/bootstrap.min.js PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/js/jquery-1.11.1.min.js PRE-CREATION 
  samza-yarn/src/main/resources/scalatev2/js/jquery.tablesorter.min.js PRE-CREATION 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnAppMasterListener.scala 6bf3046a1ae4ed8f57500acae763184084ad0e09 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/refactor/SamzaAppMasterService.scala PRE-CREATION 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/refactor/SamzaYarnAppMasterLifecycle.scala PRE-CREATION 
  samza-yarn/src/main/scala/org/apache/samza/webapp/refactor/ApplicationMasterRestServlet.scala PRE-CREATION 
  samza-yarn/src/main/scala/org/apache/samza/webapp/refactor/ApplicationMasterWebServlet.scala PRE-CREATION 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterLifecycle.scala 30cf34fe1fd3f74537d16e8a51b467cd50835357 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterService.scala 7f5d9f4af088589d4287c26737bae25567c157d7 

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


Testing
-------

Created a sample job and verified that the UI is still displayed.


Thanks,

Jagadish Venkatraman