JingsongLi commented on PR #9347: URL: https://github.com/apache/paimon/pull/9347#issuecomment-5386753854
Although the option claims to be `create-time-only` or `immutable`, the general `ALTER` validation only checks for immutability when an existing snapshot is present (`SchemaManager:354`). Consequently, after table creation but before the first snapshot, one can toggle `field-id.one-based` between `false` and `true`; while the field IDs themselves are not reordered, the schema option is overwritten, resulting in a permanent inconsistency where the configuration specifies "one-based" indexing while the actual schema still includes ID 0. New tests have even codified this behavior. Changes to the actual value should be unconditionally prohibited, just as they are for the `type` attribute. -- 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]
