[GitHub] [spark] LuciferYang commented on pull request #41230: [SPARK-43586][SQL] Use the smaller value of `Range.numElements` and `Range.numSlices` as `numSlices` of `RangeExec`

["LuciferYang (via GitHub)" <gi...@apache.org>]

LuciferYang commented on PR #41230:
URL: https://github.com/apache/spark/pull/41230#issuecomment-1559702584

   > I feel it's better to always respect the user-specified num slice parameter. If the num slice is not specified, I agree that we can make it not larger than the num elements.
   
   This may require adding a status to record whether it is a user-specified slice or use default? Sounds like it will increase the complexity of the code. I don't think it's worth because this just a minor case, maybe keep it as it is is better
   


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