Line 337 in invoke() has the following comment:
// if it's AM shut down, do not decrement maxClientRetry as we wait for
// AM to be restarted.
Ideally it should be:
// If its AM shut down, do not decrement maxClientRetry while we wait
// for its AM to be restarted.
Thanks for the patch, Alison Yu! Committed to trunk.
In the future let's remember to wait for the precommit bot to run. This typo fix looks quite safe, but having contributors press "Submit Patch" and look at the precommit output helps them get used to our workflow.
Good point. I rushed it since I knew it had no chance of failing.
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10910 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10910/)
YARN-4395. Typo in comment in ClientServiceDelegate (Contributed by (templedf: rev 6d8b4f6c2791f861a55ac78c2950f783693e912a)