You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/11/11 19:22:45 UTC

[GitHub] [spark] xkrogen commented on pull request #33446: [SPARK-36215][SHUFFLE] Add logging for slow fetches to diagnose external shuffle service issues

xkrogen commented on pull request #33446:
URL: https://github.com/apache/spark/pull/33446#issuecomment-966562874


   @Ngone51 do you have any thoughts on what @otterc shared above?
   I agree at least to the point that it adds extra complexity which may be unnecessary. Auto-configuration is great, but it will also be much harder to get it right, and this is mainly targeted at administrators/power-users who should be able to set up these configs on their own.


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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