liupc commented on issue #24462: [SPARK-26268][CORE] Do not resubmit tasks when 
executors are lost
URL: https://github.com/apache/spark/pull/24462#issuecomment-487382583
 
 
   @gczsjdy I understand this use case, it's a common need, but the current 
pluggable implementation is too flexible, and the whole impl of the shuffle 
manager are customed, but this PR is so highly relies on the kind of shuffle 
manager which must be external, that's why I think we should not do this in 
master branch, or we should put them together.
   what I mean by 'special' is that without the impl of external shuffle 
manager in master branch, It's odd to add these config alone and it's not 
commonly used with default shuffle impl.
   
   So maybe the suggestion of @squito is the good way to do it.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to