Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9182#issuecomment-152781453
So my whole feedback about the IDs wasn't because of the type, it was
because if you could use the existing values, you wouldn't need to add all the
code related to plumbing the values from the ApplicationMaster or wherever
else, and you wouldn't need that `bindToYarn` method you're adding.
If you really need the full attempt ID for the service you're adding later,
then you still need all that code, and it doesn't matter whether you expose a
String or the actual type.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
