Dandandan commented on code in PR #24456:
URL: https://github.com/apache/datafusion/pull/24456#discussion_r3824504729


##########
datafusion/common/src/config.rs:
##########
@@ -1672,6 +1672,22 @@ config_namespace! {
         /// query is used.
         pub join_reordering: bool, default = true
 
+        /// When set to true, the physical plan optimizer enumerates join 
orders for
+        /// subtrees of joins and picks the cheapest from cardinality 
estimates,
+        /// considering bushy shapes as well as left-deep ones. Subtrees whose 
inputs
+        /// lack row count statistics are left untouched.
+        pub join_enumeration: bool, default = true
+
+        /// How much cheaper an enumerated join order must be, in percent, 
before it

Review Comment:
   Yeah - my goal would be to mostly bring this down as well (i.e. remove or 
accept the regression).



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