-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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.
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
[WIP] Try to use new analyzer in EXPLAIN SYNTAX #63402
base: master
Are you sure you want to change the base?
Conversation
This is an automated comment for commit c027cf2 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
Is there a plan to re-enable EXPLAIN SYNTAX with new analyzer is used? |
@SuzyWangIBMer what do you mean? I'm gonna try to use new analyzer in |
#61059 this pr set allow_experimental_analyzer to false with "EXPLAIN SYNTAX", explain syntax does not reflect analyzer optimized query result. |
3f4a21a
to
f583073
Compare
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Documentation entry for user-facing changes
Modify your CI run
NOTE: If your merge the PR with modified CI you MUST KNOW what you are doing
NOTE: Checked options will be applied if set before CI RunConfig/PrepareRunConfig step
Include tests (required builds will be added automatically):
Exclude tests:
Extra options:
Only specified batches in multi-batch jobs: