Skip to content

Add opt-in physical row indices via rowIndex symbol - #176

Merged
platypii merged 1 commit into
masterfrom
feat/row-index-symbol
Sep 6, 2026
Merged

Add opt-in physical row indices via rowIndex symbol#176
platypii merged 1 commit into
masterfrom
feat/row-index-symbol

Conversation

@platypii

@platypii platypii commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Filtered reads compact the returned rows, leaving consumers unable to recover their physical positions in the source file. Add opt-in includeRowIndex: true and an exported rowIndex symbol so callers can read the absolute, zero-based position with row[rowIndex] while retaining filter pushdown.

The property is non-enumerable and read-only, survives filtering, projection, page pruning, and ordered parquetQuery reads, and is omitted by JSON serialization and object spread. The option defaults to false and requires object output. Includes the ParquetRow type and usage documentation.

Validation: all 662 tests pass, including physical positions across ranges, row groups, filtering, projection, page pruning, and ordered queries. ESLint and tsc --noEmit pass.

@platypii
platypii merged commit 6673b17 into master Sep 6, 2026
6 checks passed
@platypii
platypii deleted the feat/row-index-symbol branch September 6, 2026 05:51
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