fix(sqllab): show truncated table list warning by YinkaMetrics · Pull Request #41617 · apache/superset

YinkaMetrics · 2026-07-01T08:08:20Z

SUMMARY

TableSelector now uses the existing hasMore value returned by useTables to show a helper message when the API reports more tables than were returned in the current response. This makes truncated table lists visible to users and prompts them to refine their search.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Not included. The UI change is a small helper message below the table selector:

Some tables are not shown. Refine your search.

TESTING INSTRUCTIONS

cd superset-frontend
npm run test -- src/components/TableSelector/TableSelector.test.tsx
npx prettier --check src/components/TableSelector/index.tsx src/components/TableSelector/TableSelector.test.tsx
npx oxlint --config oxlint.json --quiet src/components/TableSelector/index.tsx src/components/TableSelector/TableSelector.test.tsx
npm run plugins:build
npm run type
prek run --all-files

Note: prek run --all-files passed the frontend hooks relevant to this change, including prettier (frontend), oxlint (frontend), custom rules (frontend), and Type-Checking (Frontend). The repo-wide run still failed on unrelated existing/local-environment items: mypy errors in tests/unit_tests/semantic_layers/mapper_test.py, missing yarn for docs eslint, missing helm-docs, existing Python ruff findings, and local python command resolution for two Python hooks.

ADDITIONAL INFORMATION

Has associated issue: Fixes [Bug] TableSelector silently truncates table list when DB has more tables than API page limit #40407
Required feature flags:
Changes UI
Includes DB Migration (follow approval process in SIP-59)
- Migration is atomic, supports rollback & is backwards-compatible
- Confirm DB migration upgrade and downgrade tested
- Runtime estimates and downtime expectations provided
Introduces new feature or API
Removes existing feature or API

bito-code-review · 2026-07-01T11:16:44Z

Code Review Agent Run #4eed2c

Actionable Suggestions - 0

Review Details

Files reviewed - 2 · Commit Range: 4f4a183..4f4a183
- superset-frontend/src/components/TableSelector/TableSelector.test.tsx
- superset-frontend/src/components/TableSelector/index.tsx
Files skipped - 0
Tools
- Whispers (Secret Scanner) - ✔︎ Successful
- Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

/review - Manually triggers a full AI review.
/pause - Pauses automatic reviews on this pull request.
/resume - Resumes automatic reviews.
/resolve - Marks all Bito-posted review comments as resolved.
/abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by

rusackas · 2026-07-05T06:17:16Z

Thanks @YinkaMetrics! Of the three open PRs for #40407, I think this is the one we should carry forward... it has a test and reuses the existing .table-length styling.

One thing I flagged on #40656 applies here too: the message fires purely on hasMore, so in a readOnly control (like the DatasourceEditor in non-edit mode) it tells people to refine a search they can't run. Mind gating it on the same condition the Select already uses?

The jest failures look unrelated to this change... a rebase on current master should tell us for sure. I'll update the branch now.

bito-code-review · 2026-07-05T09:25:02Z

Code Review Agent Run #b006b2

Actionable Suggestions - 0

Additional Suggestions - 1

superset-frontend/src/components/TableSelector/index.tsx - 1
- Test coverage gap · Line 343-343
  
  The fix correctly aligns message visibility with Select disabled state. However, consider adding a test case for when `disabled` is true due to missing schema (without `readOnly`) to prevent future regression.

Review Details

Files reviewed - 2 · Commit Range: 4f4a183..fc1079c
- superset-frontend/src/components/TableSelector/TableSelector.test.tsx
- superset-frontend/src/components/TableSelector/index.tsx
Files skipped - 0
Tools
- Whispers (Secret Scanner) - ✔︎ Successful
- Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

/review - Manually triggers a full AI review.
/pause - Pauses automatic reviews on this pull request.
/resume - Resumes automatic reviews.
/resolve - Marks all Bito-posted review comments as resolved.
/abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by

bito-code-review · 2026-07-05T13:03:00Z

Code Review Agent Run #7cb0db

Actionable Suggestions - 0

Review Details

Files reviewed - 1 · Commit Range: fc1079c..50ece80
- superset-frontend/src/components/TableSelector/TableSelector.test.tsx
Files skipped - 0
Tools
- Whispers (Secret Scanner) - ✔︎ Successful
- Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

/review - Manually triggers a full AI review.
/pause - Pauses automatic reviews on this pull request.
/resume - Resumes automatic reviews.
/resolve - Marks all Bito-posted review comments as resolved.
/abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by

codecov · 2026-07-05T22:09:16Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.69%. Comparing base (dc739e4) to head (50ece80).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #41617   +/-   ##
=======================================
  Coverage   64.69%   64.69%           
=======================================
  Files        2686     2686           
  Lines      148602   148605    +3     
  Branches    34288    34290    +2     
=======================================
+ Hits        96136    96139    +3     
  Misses      50701    50701           
  Partials     1765     1765

Flag	Coverage Δ
javascript	`69.55% <100.00%> (+<0.01%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

fix(sqllab): show truncated table list warning

4f4a183

pull-request-size Bot added the size/S label Jul 1, 2026

YinkaMetrics marked this pull request as ready for review July 1, 2026 08:10

dosubot Bot added change:frontend Requires changing the frontend sqllab Namespace | Anything related to the SQL Lab labels Jul 1, 2026

sadpandajoe requested review from richardfogaca, rusackas and sadpandajoe July 1, 2026 17:29

rusackas mentioned this pull request Jul 5, 2026

fix: show truncated table list message #40656

Closed

Merge branch 'master' into yinka/table-selector-has-more-warning

039b778

rusackas mentioned this pull request Jul 5, 2026

fix: show truncation notice when table list exceeds API page limit (#40407) #41485

Closed

fix(sqllab): hide table warning when search disabled

fc1079c

pull-request-size Bot added size/M and removed size/S labels Jul 5, 2026

test(sqllab): cover disabled table warning state

50ece80

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(sqllab): show truncated table list warning#41617

fix(sqllab): show truncated table list warning#41617
YinkaMetrics wants to merge 4 commits into
apache:masterfrom
YinkaMetrics:yinka/table-selector-has-more-warning

YinkaMetrics commented Jul 1, 2026

Uh oh!

bito-code-review Bot commented Jul 1, 2026 •

edited

Loading

Code Review Agent Run #4eed2c

Uh oh!

rusackas commented Jul 5, 2026

Uh oh!

bito-code-review Bot commented Jul 5, 2026 •

edited

Loading

Code Review Agent Run #b006b2

Uh oh!

bito-code-review Bot commented Jul 5, 2026 •

edited

Loading

Code Review Agent Run #7cb0db

Uh oh!

codecov Bot commented Jul 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

YinkaMetrics commented Jul 1, 2026

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

Uh oh!

bito-code-review Bot commented Jul 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Agent Run #4eed2c

Uh oh!

rusackas commented Jul 5, 2026

Uh oh!

bito-code-review Bot commented Jul 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Agent Run #b006b2

Uh oh!

bito-code-review Bot commented Jul 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Agent Run #7cb0db

Uh oh!

codecov Bot commented Jul 5, 2026

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

bito-code-review Bot commented Jul 1, 2026 •

edited

Loading

bito-code-review Bot commented Jul 5, 2026 •

edited

Loading

bito-code-review Bot commented Jul 5, 2026 •

edited

Loading