kw2542 opened a new pull request #1428:
URL: https://github.com/apache/samza/pull/1428


   Issues: app.main.class is only set for Beam apps which causes different 
workflow on AM when launching a job.
   Changes:
   1. Introduce DefaultApplicationMain to capture launch workflow for High/Low 
level jobs so on AM, all jobs are launched in the same way: 
ClusterBasedJobCoordinatorRunner#main -> app.main.class -> 
JobCoordinatorLaunchUtil
   2. Update ApplicationConfig#getAppMainClass to default to 
DefaultApplicationMain
   
   Tests:
   1. Unit Tests
   2. Deployed hello samza job successfully with the change following 
instructions on http://samza.apache.org/startup/hello-samza/latest/
   
   API Changes: None
   Upgrade Instructions: None
   Usage Instructions: None


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to