wangyum commented on code in PR #59027:
URL: https://github.com/apache/spark/pull/59027#discussion_r4256942435


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -1169,6 +1169,23 @@ object SQLConf {
     .bytesConf(ByteUnit.BYTE)
     .createWithDefaultString("10MB")
 
+  val BROADCAST_RANGE_JOIN_ENABLED = 
buildConf("spark.sql.join.broadcastRangeJoin.enabled")
+    .doc("When true, a join whose condition is a range predicate is planned as 
a " +
+      "broadcast range join instead of a broadcast nested loop join, provided 
that a " +
+      "supported build side qualifies for broadcast by size or by a BROADCAST 
hint. " +
+      "Supported join types are inner, left outer, right outer, left semi, and 
left anti. " +
+      "The preserved side is streamed. Full outer stays a broadcast nested 
loop join. " +
+      "Point-in-range and interval overlap (a.lo < b.hi AND b.lo < a.hi) 
broadcast " +
+      "an interval index. A single inequality broadcasts a sorted point index. 
" +
+      "Only types the index compares with the join predicate are recognized. " 
+
+      "A non-deterministic key stays a nested loop join. " +

Review Comment:
   Fixed: Updated the configuration doc to state that a non-deterministic 
expression is not used as a range key while eligible deterministic conjuncts 
can still select range join. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to