Skip to content

Conversation

@cj-zhukov
Copy link
Contributor

Which issue does this PR close?

Rationale for this change

This PR is for consolidating all the query_planning examples (analyzer_rule, expr_api, optimizer_rule, parse_sql_expr, plan_to_sql, planner_api, pruning, thread_pools) into a single example binary. We are agreed on the pattern and we can apply it to the remaining examples

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@cj-zhukov
Copy link
Contributor Author

High-Level Overview

This PR consolidates all query_planning examples (analyzer_rule, expr_api, optimizer_rule, parse_sql_expr, plan_to_sql, planner_api, pruning, thread_pools) into a single example binary.
Previously, each example had its own file, but now they can be executed via subcommands using:

cargo run --example query_planning -- [analyzer_rule|expr_api|optimizer_rule|parse_sql_expr|plan_to_sql|planner_api|pruning|thread_pools]

Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cj-zhukov

@comphead comphead added this pull request to the merge queue Nov 15, 2025
Merged via the queue into apache:main with commit 3863267 Nov 15, 2025
28 checks passed
@cj-zhukov cj-zhukov deleted the cj-zhukov/consolidate-examples-query-planning branch November 16, 2025 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants