JingsongLi commented on PR #10101: URL: https://github.com/apache/paimon/pull/10101#issuecomment-5832452306
Re-reviewed head `24ee618` for production. The V2 streamed file-index container has clear value for payloads above 2 GiB. The latest commit restores the old `FileIndexFormat` factory methods, public writer/reader constructors, `Reader.readAll()`, and the `int` JVM descriptor of `FileIndexMeta.sizeInBytes()`. The new compatibility test compiles an old client against the old signatures and successfully invokes it with the new runtime, addressing my previous P1 ABI finding. The V1 default remains available; V2 still needs the documented reader-first rollout. Local JDK 8 verification passed: common format and compiled-old-client compatibility suites 12/12; core index writer, rewrite processor, and system table suites 18/18, including the streamed writer path. `git diff --check` passed. Exact-head JDK 8/11, Flink, Spark, E2E, docs and licensing CI are all green. I found no remaining production blocker in the changed paths. -- 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]
