Skip to content

Conversation

@wzhero1
Copy link

@wzhero1 wzhero1 commented Dec 22, 2025

Purpose

This PR adds support for accessing system fields (specifically sequence_number) in AuditLogTable and BinlogTable.

Currently, downstream consumers often need the sequence_number to ensure strict ordering or to implement exactly-once processing when reading from audit logs or binlogs. However, these system tables previously did not expose this field.

This change implements KeyValueSystemFieldsRecordReader to enable the projection of system fields for KeyValue records, allowing users to query sequence_number directly from audit log and binlog tables.

Tests

  • Added IncrementalReadSystemFieldsTest to verify functionality.
  • Updated AuditLogTableTest and BinlogTableTest to cover sequence_number read cases.

API and Format

  • No change to storage format.
  • No public API change.

Documentation

  • No documentation change required.

@wzhero1 wzhero1 force-pushed the auditlog-support-seqnum branch 8 times, most recently from aff70b7 to ed453a8 Compare December 24, 2025 11:42
@wzhero1 wzhero1 force-pushed the auditlog-support-seqnum branch from ed453a8 to eae1a70 Compare December 25, 2025 03:40
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.

1 participant