-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
*: fix a bug causes indexed virtual generated column return wrong value and refine admin check table (#18408) #19062
*: fix a bug causes indexed virtual generated column return wrong value and refine admin check table (#18408) #19062
Conversation
Please fix the CI |
…rtual_index_40hotfix
3c47651
to
31ea1b5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Any conflicts? |
/merge |
Cherry-pick from a middle commit. Because the previous commit contains clustered indexes stuffs. |
/run-all-tests |
@wjhuang2016 merge failed. |
/run-unit-test |
What problem does this PR solve?
Issue Number: close #17989
Problem Summary:
column substitution
to decode a multi-level virtual generated column, which is buggy.admin check table
uses tableReader, and tableReader can output rows with a virtual generated column already, we don't need to compute it again. Besides, the previous implementation also suffered the above problem.I’m
What is changed and how it works?
ColumnInfos2ColumnsAndNames
.genExprs
incheckIndexValue
.compareData
, don't compute the virtual generated column again.rewriteAstExpr
, so that we can userewrite
in package expression.buildPhysicalIndexLookUpReader
, make it simple.table/tables/gen_expr_test.go → util/generatedexpr/gen_expr_test.go
. to avoid circular reference.What's Changed:
How it Works:
Related changes
Check List
Tests
Side effects
Release note