Skip to content

Add V2 batch format with statistics collection#2886

Open
platinumhamburg wants to merge 1 commit intoapache:mainfrom
platinumhamburg:filter-v2-batch-format
Open

Add V2 batch format with statistics collection#2886
platinumhamburg wants to merge 1 commit intoapache:mainfrom
platinumhamburg:filter-v2-batch-format

Conversation

@platinumhamburg
Copy link
Contributor

Introduce V2 batch format that collects min/max statistics for each column to enable efficient filtering.

  • Add LogRecordBatchStatistics and related classes for statistics collection
  • Add StatisticsConfigUtils for parsing table.statistics.columns configuration
  • Extend DefaultLogRecordBatch to support V2 format with statistics
  • Place statistics data between header and records with StatisticsLength field
  • Add comprehensive tests for statistics collection and parsing

Purpose

Linked issue: close #2885

Brief change log

Tests

API and Format

Documentation

@platinumhamburg platinumhamburg force-pushed the filter-v2-batch-format branch 5 times, most recently from af26717 to 836948c Compare March 17, 2026 04:38
@wuchong wuchong requested a review from Copilot March 21, 2026 08:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a new V2 Arrow log record batch format that can embed per-column min/max + null-count statistics between the batch header and record payload, enabling future filter pushdown improvements (Issue #2885 / FIP-10).

Changes:

  • Add statistics collection/serialization/parsing APIs and implementations (collector/writer/parser + batch-access API).
  • Extend Arrow log batch building, reading, and projection logic to account for a V2 layout with an optional statistics section and a StatisticsLength header field.
  • Add table-level configuration (table.statistics.columns) utilities and validation, and wire statistics collection into the client write path.

Reviewed changes

Copilot reviewed 32 out of 32 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
fluss-server/src/main/java/org/apache/fluss/server/kv/wal/ArrowWalBuilder.java Adapts builder call sites to new statistics-capable Arrow batch builder signature.
fluss-server/src/main/java/org/apache/fluss/server/coordinator/CoordinatorService.java Validates table statistics configuration on CREATE TABLE.
fluss-common/src/test/java/org/apache/fluss/record/TestData.java Adds schemas/data for statistics-related tests.
fluss-common/src/test/java/org/apache/fluss/record/MemoryLogRecordsArrowBuilderTest.java Extends batch builder tests to cover V2 + statistics.
fluss-common/src/test/java/org/apache/fluss/record/LogRecordBatchStatisticsTestUtils.java Adds reusable utilities for generating batches with statistics in tests.
fluss-common/src/test/java/org/apache/fluss/record/LogRecordBatchStatisticsTest.java Adds end-to-end tests for statistics extraction/caching across batch impls.
fluss-common/src/test/java/org/apache/fluss/record/LogRecordBatchStatisticsParserTest.java Adds tests for statistics parsing/validation helpers.
fluss-common/src/test/java/org/apache/fluss/record/LogRecordBatchStatisticsCollectorTest.java Adds tests for collector behavior across types/nulls/mappings.
fluss-common/src/test/java/org/apache/fluss/record/LogRecordBatchFormatTest.java Adds V2 header/offset assertions and stats-aware offsets.
fluss-common/src/test/java/org/apache/fluss/config/StatisticsConfigUtilsTest.java Adds tests for statistics config validation.
fluss-common/src/main/java/org/apache/fluss/types/DataTypeChecks.java Adds isBinaryType helper used by stats config/mapping logic.
fluss-common/src/main/java/org/apache/fluss/row/arrow/ArrowWriter.java Exposes RowType schema via getSchema() for stats wiring.
fluss-common/src/main/java/org/apache/fluss/row/aligned/AlignedRow.java Adds AlignedRow.from(RowType, InternalRow) conversion helper.
fluss-common/src/main/java/org/apache/fluss/record/bytesview/MultiBytesView.java Enhances builder API (addBytes(BytesView), isEmpty) and improves file-region merging.
fluss-common/src/main/java/org/apache/fluss/record/MemoryLogRecordsArrowBuilder.java Implements V2 batch building with optional embedded statistics + CRC handling changes.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatchStatisticsWriter.java New: serializes statistics in a compact schema-aware format.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatchStatisticsParser.java New: parses/validates statistics payloads from multiple memory sources.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatchStatisticsCollector.java New: collects per-column min/max/null-counts during batch build.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatchStatistics.java New: statistics interface exposed from LogRecordBatch.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatchFormat.java Adds V2 constants/layout helpers, statistics offsets, and header mutation helper.
fluss-common/src/main/java/org/apache/fluss/record/LogRecordBatch.java Adds getStatistics(ReadContext) API to batches.
fluss-common/src/main/java/org/apache/fluss/record/FileLogProjection.java Updates projection to skip stats section for V2 and clear stats header fields.
fluss-common/src/main/java/org/apache/fluss/record/FileLogInputStream.java Adds stats parsing and provides optional “trim stats” BytesView generation for V2.
fluss-common/src/main/java/org/apache/fluss/record/DefaultLogRecordBatchStatistics.java New: zero-copy statistics view with full-schema wrappers + mapping logic.
fluss-common/src/main/java/org/apache/fluss/record/DefaultLogRecordBatch.java Updates record decoding offsets for V2 (stats-aware) and implements statistics parsing/caching.
fluss-common/src/main/java/org/apache/fluss/metadata/TableInfo.java Computes and caches stats column index mappings based on table config.
fluss-common/src/main/java/org/apache/fluss/config/TableConfig.java Adds statistics config accessors and enablement checks.
fluss-common/src/main/java/org/apache/fluss/config/StatisticsConfigUtils.java New: validates table.statistics.columns against schema + supported types.
fluss-common/src/main/java/org/apache/fluss/config/ConfigOptions.java Adds table.statistics.columns option definition/documentation.
fluss-client/src/test/java/org/apache/fluss/client/write/ArrowLogWriteBatchTest.java Updates tests for ArrowLogWriteBatch constructor signature.
fluss-client/src/main/java/org/apache/fluss/client/write/RecordAccumulator.java Wires per-table statistics collection into batch creation when enabled.
fluss-client/src/main/java/org/apache/fluss/client/write/ArrowLogWriteBatch.java Passes statistics collector into Arrow batch builder.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@platinumhamburg platinumhamburg force-pushed the filter-v2-batch-format branch from 836948c to 51dd84c Compare March 22, 2026 11:55
Introduce V2 batch format that collects min/max statistics for each column
to enable efficient filtering.

- Add LogRecordBatchStatistics and related classes for statistics collection
- Add StatisticsConfigUtils for parsing table.statistics.columns configuration
- Extend DefaultLogRecordBatch to support V2 format with statistics
- Place statistics data between header and records with StatisticsLength field
- Add comprehensive tests for statistics collection and parsing
@platinumhamburg platinumhamburg force-pushed the filter-v2-batch-format branch from 51dd84c to e595708 Compare March 23, 2026 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[common] Add V2 batch format with statistics collection for filter pushdown

2 participants