jianguotian commented on PR #70: URL: https://github.com/apache/paimon-mosaic/pull/70#issuecomment-5336933071
I found and fixed two regressions in the current head: - keep field-name/order validation, but leave type/nullability enforcement on the existing writer validation path so current error/test contracts remain intact; - report the actual mapped batch column index in the name-mismatch error. Because I cannot push to `QuakeWang:mosaic-schema-guard`, the fix is available as `fbaf0a0b5df19f5d7141e4b78ccfbc17601c1215` and helper PR https://github.com/QuakeWang/paimon-mosaic/pull/1. Validation: - local full Rust/Python/C++/Java suites, fmt and Clippy passed; - exact-head GitHub CI against current Apache main `b613dd25eba45afbc860171e109df19495ba0e3c`: all 5 jobs passed: https://github.com/jianguotian/paimon-mosaic/actions/runs/32209856324. Please merge the helper PR (or cherry-pick the commit) to update this PR. -- 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]
