kw2542 opened a new pull request #1361: URL: https://github.com/apache/samza/pull/1361
Symptom:Incorrect job status reported, which is previous deployed job status instead of current job. Cause:ClientHelper#isActiveApplication is invoking toAppStatus twice when comparing status to Running or New. It could happen that 1st time, the status is New which failed Running check, and 2nd invocation returns Running, which failed New check, this will result in the current app to be classified as non active causing ClientHelper to return job status of a previous deployed app. Changes: Update ClientHelper#isActiveApplication to invoke toAppStatus once and compare both status against this single result. Tests: None 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]
