-
Notifications
You must be signed in to change notification settings - Fork 1.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
[Feature]Support add/drop field for struct column(part3) #47217
Conversation
Signed-off-by: sevev <[email protected]>
Signed-off-by: sevev <[email protected]>
fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
Outdated
Show resolved
Hide resolved
* @throws DdlException | ||
*/ | ||
private boolean processDropField(DropFieldClause alterClause, OlapTable olapTable, | ||
Map<Long, LinkedList<Column>> indexSchemaMap, List<Index> indexes) throws DdlException { |
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.
it seems that indexes is not used in this function.
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.
it seems that indexes is not used in this function.
updated.
Signed-off-by: sevev <[email protected]>
Signed-off-by: sevev <[email protected]>
fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: sevev <[email protected]>
Signed-off-by: sevev <[email protected]>
Signed-off-by: sevev <[email protected]>
Signed-off-by: sevev <[email protected]>
Quality Gate passedIssues Measures |
[BE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[FE Incremental Coverage Report]✅ pass : 153 / 173 (88.44%) file detail
|
@mergify backport branch-3.3 |
✅ Backports have been created
|
Signed-off-by: sevev <[email protected]> (cherry picked from commit 13899a2) # Conflicts: # fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java # fe/fe-core/src/main/java/com/starrocks/sql/analyzer/AlterTableClauseAnalyzer.java # fe/fe-core/src/main/java/com/starrocks/sql/ast/AddFieldClause.java # fe/fe-core/src/main/java/com/starrocks/sql/ast/DropFieldClause.java # fe/fe-core/src/main/java/com/starrocks/sql/ast/StructFieldDesc.java # test/sql/test_add_drop_field/R/test_add_drop_field # test/sql/test_add_drop_field/T/test_add_drop_field
…7217) Signed-off-by: sevev <[email protected]>
@mergify backport branch-3.3 |
✅ Backports have been created
|
…7217) (#47497) Signed-off-by: sevev <[email protected]> Co-authored-by: zhangqiang <[email protected]>
@mergify backport branch-3.2 |
✅ Backports have been created
|
Signed-off-by: sevev <[email protected]> (cherry picked from commit 13899a2) # Conflicts: # fe/fe-core/src/main/java/com/starrocks/alter/SchemaChangeHandler.java
…7217) (#47590) Signed-off-by: sevev <[email protected]> Co-authored-by: zhangqiang <[email protected]>
Why I'm doing:
It is the third pr to support add/drop field for struct column.
What I'm doing:
Support add/drop field for table enable fast schema evolution in shared nothing mode.
Fixes #46452
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: