The GitHub Actions job "CI" on fluss.git/filter-v2-batch-format has failed.
Run started by GitHub user platinumhamburg (triggered by platinumhamburg).

Head commit for run:
bd8123b3562c10a8b4e14bb30722cd3b1908bc8f / 白鵺 <[email protected]>
Address PR #2886 review feedback for V2 batch format with statistics

Config & API:
- Change TABLE_STATISTICS_COLUMNS default from "*" to noDefaultValue()
- Introduce StatisticsColumnsConfig with three-state mode 
(DISABLED/ALL/SPECIFIED)
- Replace isBinaryType blacklist with isSupportedStatisticsType whitelist
- Update config documentation with compatibility requirements

Format versioning:
- Renumber versions: V0=base, V1=statistics, V2=leaderEpoch(future)
- Remove redundant STATISTICS_FLAG, use statisticsLength > 0 instead
- Make statisticsLengthOffset/leaderEpochOffset throw for unsupported versions
- Remove unsafe arrowChangeTypeOffset(magic) method

Write path:
- Serialize statistics directly to pagedOutputView (no temp byte[])
- Restore zero-copy CRC over contiguous memory segments
- Fix estimatedSizeInBytes to use recordBatchHeaderSize(magic)

Writer optimization:
- Replace expensive estimatedSizeInBytes with cached heuristic estimate
- Reuse AlignedRowWriter instance across writes
- Extract backing byte[] directly from MemorySegment for OutputView writes

Parser optimization:
- Replace sequential MemorySegmentInputView reads with fixed-offset access
- Add ByteBuffer.isDirect() check before wrapOffHeapMemory
- Add skipBytes() to MemorySegmentInputView

Correctness:
- Remove redundant minSet/maxSet arrays in statistics collector
- Throw UnsupportedOperationException for unsupported types in AlignedRow
- Fix minValuesSize condition from <= 0 to == 0

Code cleanup:
- Remove unused methods (getBytesView, createTrimmedBytesView, addBytes, etc.)
- Add @Nullable annotations on statisticsCollector and parseStatistics return
- Add logging in statistics parsing catch blocks
- Wrap clearStatisticsFromHeader with statisticsLength > 0 guard

Tests & docs:
- Add zero-rows and all-null column test cases
- Update ConfigOptions docs with V1 format and compatibility notes

Report URL: https://github.com/apache/fluss/actions/runs/23531841478

With regards,
GitHub Actions via GitBox

Reply via email to