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 2022/12/16 06:59:59 UTC

[GitHub] [spark] mridulm commented on pull request #39076: [SPARK-41530][CORE] Rename MedianHeap to PercentileMap and support percentile

mridulm commented on PR #39076:
URL: https://github.com/apache/spark/pull/39076#issuecomment-1354301805

   > Using the median task duration to trigger task speculation may not be the best option. It's better to allow users to configure the percentile if they want task speculation to happen more or less likely.
   
   Sounds good.
   Note that given the complexity of how we determine whether a task is speculatable  or not, it is not just a matter of turning the dial on the percentile :-)


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