AngersZhuuuu edited a comment on pull request #31048: URL: https://github.com/apache/spark/pull/31048#issuecomment-756788354
> those are just default values so the user can change what they request. Normally I would agree that keeping the same makes sense. I tried to go back to see if there was a reason it was made this way but didn't see any comments on it. > > One question is whether this is a breaking change because users that do this will now get a different result. thoughts from others? Ideally it doesn't matter as they would check what the quantiles are but if they aren't it would be different results. Yea, and default value is not show in [monitoring page](https://spark.apache.org/docs/3.0.0/monitoring.html) and user will see ui's quantiles in Web UI. They may think the are same. we need to add this default value to monitoring page too. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
