Skip to content

fix: take schema from reader instead of builder#811

Open
Droeftoeter wants to merge 1 commit intokylebarron:mainfrom
Droeftoeter:fix-fetching-individual-columns
Open

fix: take schema from reader instead of builder#811
Droeftoeter wants to merge 1 commit intokylebarron:mainfrom
Droeftoeter:fix-fetching-individual-columns

Conversation

@Droeftoeter
Copy link

When fetching a subset of columns the full schema was returned, now it returns the projected schema instead.

Fixes #810

@github-actions
Copy link

github-actions bot commented Oct 8, 2025

Asset Size Compressed Size
report_pkg/async_full/parquet_wasm_bg.wasm 6.7MB 1.3MB
report_pkg/slim/parquet_wasm_bg.wasm 4.9MB 678KB
report_pkg/sync/parquet_wasm_bg.wasm 6.2MB 1.2MB

@kylebarron
Copy link
Owner

Thanks! Would you be able to add a JS test that failed before this change but passes after it? I figure just passing a column projection into a read function should test this code path

@DServy
Copy link

DServy commented Jan 24, 2026

Any movements on getting this merged in? Just ran into this issue myself today.

@kylebarron
Copy link
Owner

Ideally I'd like a simple test for this just to ensure we don't regress in the future.

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.

Fetching a subset of columns results in error because the full schema is always returned

3 participants