zhuxiangyi commented on PR #9809: URL: https://github.com/apache/paimon/pull/9809#issuecomment-5674468568
Thanks for the review. Rebased onto master after #9818 landed (it touched `VariantGet` / `BaseVariantReader` and added tests next to the ones here); conflicts are resolved and the change is still a single commit. On the Spark 3 / Scala 2.13 failure: it was `RowTrackingTestBase` "Data Evolution: concurrent merge and small files compact" hitting an expired snapshot during a concurrent MERGE + compact, which this PR does not touch (Spark 3 does not run the Variant suites at all). CI is re-running on the rebased head and that job has not failed again so far. Locally after the rebase: `GenericVariantTest` / `VariantShreddingReadTest` pass and the four Spark 4.1 Variant suites pass (124 tests). -- 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]
