JingsongLi commented on PR #370: URL: https://github.com/apache/paimon-rust/pull/370#issuecomment-4697196606
Cross-PR note with #340: if lands before or together with this alter-table work, needs to update aggregation field-scoped options as well. Right now the rename path rewrites , , and , but it does not rename keys such as or . After renaming an aggregation value column, those options would still point at the old column name. Since the final validation here reruns blob / partial-update / first-row checks but not aggregation validation, the stale option can be persisted and the renamed column may silently fall back to the default aggregator/delimiter at read time. This is not visible against current by itself, but it should be handled if this PR is intended to compose with #340. -- 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]
