QuakeWang commented on PR #7598:
URL: https://github.com/apache/paimon/pull/7598#issuecomment-4529002963
@JingsongLi Thanks for the review.
I added explicit coverage for the side effects:
- `BundleAwareRowDataRollingFileWriterTest` verifies the sequence counter
and min/max sequence numbers after bundle writes.
- `DedicatedFormatRollingFileWriterVectorTest` verifies sequence side
effects for the normal/blob/vector fan-out path.
- It also verifies the main-file index path by enabling bitmap index on
`f0`, writing a single-use bundle, and evaluating the embedded index.
I kept `ReplayableBundleRecords` / `ProjectableBundleRecords` in this PR
because they are part of the fix: replayability is needed for safe fan-out, and
projectability keeps the Arrow bundle fast path.
The remaining failed CI is a timeout in
`SinkSavepointITCase.testRecoverFromSavepoint`, which looks unrelated to this
change.
--
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]
