jackylee-ch commented on PR #975: URL: https://github.com/apache/paimon-rust/pull/975#issuecomment-5979755210
Thanks for the review — fixed in the latest commit (CI re-running). Both the non-trailing ADD and DROP guards now compare positions among non-partition fields only, matching the physical CSV schema (partition columns are excluded, as `FormatReadBuilder` does). So on `(id, label, pt)` partitioned by `pt`, appending `extra` after `label` and dropping `label` are now accepted as the safe trailing changes they are on the physical layout `(id, label)`, while a genuine physical shift (e.g. dropping `id`) stays rejected. Added a partitioned-table test alongside the existing ADD/DROP cases. -- 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]
