Skip to content

[CALCITE-7417] Add a large plan benchmark for HepPlanner#4796

Open
zhuwenzhuang wants to merge 1 commit intoapache:mainfrom
zhuwenzhuang:union_tree_benchmark
Open

[CALCITE-7417] Add a large plan benchmark for HepPlanner#4796
zhuwenzhuang wants to merge 1 commit intoapache:mainfrom
zhuwenzhuang:union_tree_benchmark

Conversation

@zhuwenzhuang
Copy link
Contributor

Add a large plan benchmark for HepPlanner.
We can use it to identify performance bottlenecks.

@zhuwenzhuang zhuwenzhuang force-pushed the union_tree_benchmark branch 2 times, most recently from cf1b43e to c912959 Compare February 15, 2026 02:59
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the purpose of this statement?

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you expand this description a bit? What kind of plan is generated, what optimizations are tested, and what this benchmark is really measuring.

Copy link
Contributor Author

@zhuwenzhuang zhuwenzhuang Feb 16, 2026

Choose a reason for hiding this comment

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

This part is mainly used to evaluate HepPlanner’s ability to optimize large queries. Its performance will be improved step by step with optimizations such as TypeDigest, the firedRuleCache, iterator improvements, GC related optimization, etc. At the moment this benchmark is expected to be pretty slow because the main optimizations have not been committed yet; in the future we can use it as a baseline for incremental optimization.

@mihaibudiu mihaibudiu added LGTM-will-merge-soon Overall PR looks OK. Only minor things left. labels Feb 19, 2026
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments