lakshmi-manasa-g opened a new pull request #1439:
URL: https://github.com/apache/samza/pull/1439


   **Issue:** AM and container share a few string literals but these are not 
constants and if the strings do not exactly match on both, no tests fails but 
the job certainly will.
   
   **Changes:** in the existing CoordinationConstants, new constants created 
for string literals: "containerHeartbeat", "executionContainerId", 
"samza.autoscaling.server.url"
   
   **Tests:** all existing tests pass. 
   
   **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