Github user ajbozarth commented on the pull request:

    https://github.com/apache/spark/pull/12123#issuecomment-206623940
  
    I just pushed an update. I changed how I get the username so it uses the 
same method for both active and history. This method actually pulls the 
information from the same source as both sc.sparkUser and the API do at their 
root, the system env. Except with this method we make sure we always get the 
correct user; I noticed a race condition when using sc.sparkUser while finding 
this solution. After starting up the master when two spark applications are 
started at once, one by the user who started the master and one who didn't, 
both would appear to been started by the user who started the master. This 
would only happened on the first apps started, any apps started after would 
behave as expected. This race condition doesn't exist with my change.


---
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

Reply via email to