Github user ajbozarth commented on the issue:

    https://github.com/apache/spark/pull/17795
  
    This is a clean way of doing this, but my reasoning behind how I did it was 
from my understanding even if there's only one attempt you could still access 
it with an attempt id, I actually went into the code to check that it worked 
that way. I'm just wondering if there's a need to add this if it works as is, 
but if the way I had it originally is just a loophole I'm fine changing it.


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