[GitHub] [spark] holdenk commented on a diff in pull request #37819: [SPARK-40377][SQL] Allow customize maxBroadcastTableBytes and maxBroadcastRows

[GitBox <gi...@apache.org>]

holdenk commented on code in PR #37819:
URL: https://github.com/apache/spark/pull/37819#discussion_r989487173


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -518,6 +518,24 @@ object SQLConf {
     .bytesConf(ByteUnit.BYTE)
     .createWithDefaultString("10MB")
 
+  val MAX_BROADCAST_TABLE_BYTES =
+    buildConf("spark.sql.maxBroadcastTableBytes")
+      .doc("Configures the maximum size in bytes for a table when performing a broadcast join. " +
+        "The maximum allowed value is 8G.")

Review Comment:
   Any particular reason why this value was chosen?



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