LuciferYang opened a new pull request, #13448: URL: https://github.com/apache/gravitino/pull/13448
### What changes were proposed in this pull request? When a bundled alter updates a column without an explicit `UpdateColumnPosition`, `applyColumnChanges` now re-inserts the column at the index it was removed from, rather than at the column's stored position. The explicit-position path is unchanged. ### Why are the changes needed? The stored position equals a live list index only before any sibling add/delete in the same alter. So a delete-then-update bundle threw `IndexOutOfBoundsException`, and an add-then-update bundle silently reordered the columns. Fix: #13447 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Added `TestManagedTableOperations.testAlterTableBundledColumnChanges`, covering the delete-then-update and add-then-update bundles. It fails against the pre-fix code. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
