villebro commented on code in PR #2765:
URL:
https://github.com/apache/datafusion-ballista/pull/2765#discussion_r4471619184
##########
ballista/core/src/config.rs:
##########
@@ -87,6 +87,10 @@ pub const BALLISTA_CLIENT_IO_RETRY_WAIT_TIME_MS: &str =
"ballista.client.io_retry_wait_time_ms";
/// Enables adaptive query planning
pub const BALLISTA_ADAPTIVE_PLANNER_ENABLED: &str =
"ballista.planner.adaptive.enabled";
+/// Lets the adaptive planner run a repeated subplan's stage once and have
+/// every consumer read the same shuffle output (Spark's `ReuseExchange`).
+pub const BALLISTA_ADAPTIVE_REUSE_EXCHANGE: &str =
+ "ballista.planner.adaptive.reuse_exchange";
Review Comment:
If we're planning on making this work apply to other planners, too, then
@comphead's suggestion of going with `ballista.planner.reuse_exchange.enabled`
makes more sense so we don't have to rename later.
--
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]
