lxy-9602 commented on PR #8392: URL: https://github.com/apache/paimon/pull/8392#issuecomment-4900166064
> This PR has become too large, can we split some PRs out. Thanks for the detailed review. To make the changes easier to review, I have split the original PR into smaller independent parts: 1. Refactor the Variant read path to use `ShreddingReadPlan`, so Variant materialization is no longer tightly coupled with the Parquet reader (this pr). 2. Add MAP shared-shredding schema validation and common utility updates (#8487). 3. Add append-table MAP shared-shredding read/write support, including the full append E2E test coverage. This should make each part more focused and avoid mixing framework refactoring, validation, and the MAP shared-shredding read/write implementation in one large PR. Would be grateful if you could take another look when you have time. -- 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]
