Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/12075#issuecomment-217768123
@srowen , currently we assume attempt id is `None` when spark application
is running on yarn client mode. This assumption is used not only in REST api,
but also in history server and yarn extension services.
1. Only changing here will break the consistency for other parts as I
mentioned before.
2. Even if we address all the parts related to attempt id, still we may
break the backward compatibility, especially for event log file name.
So IMHO I suggest not to change the behavior of attempt id unless we have a
sufficient reason.
---
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
