Github user steveloughran commented on the pull request:
https://github.com/apache/spark/pull/9182#issuecomment-152536617
OK, new patch with uses String values for appId and app attempt ID in the
service extension binding âbut explicitly derives these from the YARN app &
attempt IDs, so that services and downstream code can be confident they will be
locatable in the YARN services (RM, NM , etc).
Tests (here and dependent pull requests) all updated to match.
I've moved the Stub app/attempd ID classes out, as they are now only needed
for integration tests in the final history provider patch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
