[GitHub] srowen commented on issue #23871: [SPARK-23433][SPARK-25250] [CORE] Later created TaskSet should learn about the finished partitions

[GitBox <gi...@apache.org>]

srowen commented on issue #23871: [SPARK-23433][SPARK-25250] [CORE] Later created TaskSet should learn about the finished partitions 
URL: https://github.com/apache/spark/pull/23871#issuecomment-466524639
 
 
   I get where you're coming from @pgandhi999 . I'd read this as part of one larger discussion, and indeed it happens sometimes that the easiest way to consider an alternative is to take a look at it concretely. I don't read this as a hijack or anything.
   
   I haven't read the long discussion on the other PR but can see it's a tricky change and you have a couple people with deep experience working on it with you to get it right. It may be that you cross-pollinate ideas between the two PRs. It doesn't so much matter which one gets merged as long as it's a consensus and the best we can come up with.
   
   If the issue is credit then obviously anyone who reads these knows you've driven a lot of the discussion on the important change and this is based on your change.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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