Skip to content

feat: Optimize statement_info exec_plan storage with conditional Physical Plan output #23387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
XuPeng-SH wants to merge 16 commits into
base: main
Choose a base branch
from
Open

feat: Optimize statement_info exec_plan storage with conditional Physical Plan output #23387

XuPeng-SH wants to merge 16 commits into from

Conversation

@XuPeng-SH
Copy link
Contributor

@XuPeng-SH XuPeng-SH commented Dec 28, 2025

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #23386

What this PR does / why we need it:

Solution:

  • Default: Overview only (~500 bytes) for execution_time ≤ (5s + longQueryThreshold) AND ≤ (3x longQueryThreshold)
  • Detailed: Full Physical Plan for truly slow queries

@matrix-meow matrix-meow added the size/XL Denotes a PR that changes [1000, 1999] lines label Dec 28, 2025
@XuPeng-SH XuPeng-SH requested a review from fengttt as a code owner December 28, 2025 09:15
@matrix-meow matrix-meow added size/XXL Denotes a PR that changes 2000+ lines and removed size/XL Denotes a PR that changes [1000, 1999] lines labels Dec 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aunjgr aunjgr aunjgr approved these changes

@zhangxu19830126 zhangxu19830126 zhangxu19830126 approved these changes

@ouyuanning ouyuanning ouyuanning approved these changes

@aptend aptend aptend approved these changes

@heni02 heni02 heni02 approved these changes

@fengttt fengttt Awaiting requested review from fengttt fengttt is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

kind/enhancement size/XXL Denotes a PR that changes 2000+ lines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@XuPeng-SH @aunjgr @zhangxu19830126 @ouyuanning @aptend @heni02 @matrix-meow