spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From squito <...@git.apache.org>
Subject [GitHub] spark issue #20462: [SPARK-23020][core] Fix another race in the in-process l...
Date Fri, 02 Feb 2018 20:44:47 GMT
Github user squito commented on the issue:

    https://github.com/apache/spark/pull/20462
  
    lgtm
    
    The fix here makes sense to me, I see how it breaks the test.  I'm just wondering, do
we need to doc this at all for users, eg. just clearly describe it in jira?  I realize most
users will never hit this, as its only super short apps, but just say that its possible for
very short apps, they never enter the FINISHED state but instead go to LOST, even though the
app finished successfully?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message