[GitHub] [samza] dxichen opened a new pull request, #1602: SAMZA-2737: Change RemoteAppRunner to plan for local deployments

[GitBox <gi...@apache.org>]

dxichen opened a new pull request, #1602:
URL: https://github.com/apache/samza/pull/1602

   Changes: 
   - Add checks in RemoteApplicationRunner to proceed with the legacy planning step if we are using ProcessJob or ThreadJob
   - Remove the planning from the processJob and ThreadJob
   
   Tests: Tested with local job
   
   API changes: None
   
   Upgrade instructions: None
   
   Usage instructions: if implementing CheckpointManager then can either leave the readAllCheckpoints method untouched or override to read from the implementation-specific location to fetch all checkpoints and return last recorded for all tasks (incl those not in the current job model)
   
   Backwards compatible: yes. Does not change any existing interface methods


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [samza] xinyuiscool merged pull request #1602: SAMZA-2737: Change RemoteAppRunner to plan for local deployments

[GitBox <gi...@apache.org>]

xinyuiscool merged PR #1602:
URL: https://github.com/apache/samza/pull/1602


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org