dongjoon-hyun opened a new pull request, #41234:
URL: https://github.com/apache/spark/pull/41234

   ### What changes were proposed in this pull request?
   
   This is a backporting of #41232 
   
   This PR aims to fix `cannotBroadcastTableOverMaxTableBytesError` to use 
`bytesToString` instead of shift operations.
   
   ### Why are the changes needed?
   
   To avoid user confusion by giving more accurate values. For example, 
`maxBroadcastTableBytes` is 1GB and `dataSize` is `2GB - 1 byte`.
   
   **BEFORE**
   ```
   Cannot broadcast the table that is larger than 1GB: 1 GB.
   ```
   
   **AFTER**
   ```
   Cannot broadcast the table that is larger than 1024.0 MiB: 2048.0 MiB.
   ```
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes, but only error message.
   
   ### How was this patch tested?
   
   Pass the CIs with newly added test case.


-- 
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-unsubscr...@spark.apache.org

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


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

Reply via email to