QuakeWang commented on PR #681:
URL: https://github.com/apache/paimon-rust/pull/681#issuecomment-5311142503

   @JingsongLi Thanks, confirmed. The nested `_BUILD_SCHEMA_ID` extension is 
not backward compatible with the current Rust fast decoder. I see two safe 
options: (1) remove build-schema provenance from this PR and use the 
conservative schema-history fallback, which preserves the current Java format 
but cannot distinguish indexes rebuilt after a type change from legacy indexes; 
or (2) store it as a trailing nullable top-level `IndexManifestEntry` field, 
which the current main reader already skips while retaining the 
ALTER/index-commit race guard. Would you prefer removing the field from this 
PR, or is the top-level optional extension acceptable?


-- 
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]

Reply via email to