feat: Improve Table Previews - #2593
Open
camielvs wants to merge 1 commit into
Open
Conversation
🎩 PreviewA preview build has been created at: |
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jul 29, 2026
camielvs
force-pushed
the
07-29-feat_parquet_improvements_2
branch
from
July 30, 2026 00:19
e484306 to
360e7b9
Compare
camielvs
force-pushed
the
07-21-feat_improve_parquet_viewer
branch
from
July 30, 2026 00:19
436b6ef to
c472ea6
Compare
camielvs
force-pushed
the
07-29-feat_parquet_improvements_2
branch
from
July 30, 2026 00:46
360e7b9 to
5613e32
Compare
camielvs
force-pushed
the
07-21-feat_improve_parquet_viewer
branch
2 times, most recently
from
July 30, 2026 14:57
d3eb937 to
99d7a62
Compare
camielvs
force-pushed
the
07-29-feat_parquet_improvements_2
branch
2 times, most recently
from
July 30, 2026 16:02
1abf10f to
f8a11f0
Compare
camielvs
marked this pull request as ready for review
July 30, 2026 16:24
camielvs
force-pushed
the
07-29-feat_parquet_improvements_2
branch
2 times, most recently
from
July 30, 2026 17:54
9d07e68 to
2c40fb0
Compare
camielvs
force-pushed
the
07-21-feat_improve_parquet_viewer
branch
from
July 30, 2026 18:23
99d7a62 to
ec1e716
Compare
camielvs
force-pushed
the
07-29-feat_parquet_improvements_2
branch
from
July 30, 2026 18:23
2c40fb0 to
227a763
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Description
Follow-up to #2557. This PR adds extra viewer affordances for both Parquet and CSV/TSV.
1. Load more / Load max (Parquet). The base PR previews the top 100 rows and stops. Here the viewer can pull additional pages on demand via range reads — Load more fetches the next batch, Load max fills up to the preview cap. Only the newly requested range is fetched each time; rows already loaded are never re-read, and the whole file is never downloaded.
2. Preview limit is a cell budget, not a row count. The preview table renders every cell into the DOM (no virtualization), so its cost scales with rows × columns, not rows alone. A flat 1,000-row cap therefore over-protects narrow tables and under-protects wide ones. Instead, the preview is bounded by a 50,000-cell budget: the row limit is
floor(50,000 / columnCount), with an absolute backstop of 10,000 rows so a very narrow table still can't flood the DOM. This adapts to table shape:Rows are atomic: the budget is floored to whole rows up front, so the rendered cell count never exceeds 50,000 and no partial rows are shown. The same limit applies to both Parquet (Load max) and CSV/TSV previews.
3. Download full dataset. When the preview limit is reached but the file still has more rows than are shown, the footer surfaces a Download full dataset link so the user has a clear escape hatch to the complete data (opens the signed URL for remote artifacts).
4. CSV/TSV parity. CSV/TSV previews now report the exact total row count and column count in the header, matching Parquet. Counting is done with a streaming parse (every row is counted, only the preview rows are retained). Remote CSVs get a Download full dataset link to the signed URL; inline CSV values trigger a direct file download.
Related Issue and Pull requests
Type of Change
Checklist
Test Instructions
Additional Comments