Skip to content

fix: Restrict DrillDetailTable pageSizeOptions to only valid option (50)#37733

Open
RamiNoodle733 wants to merge 1 commit intoapache:masterfrom
RamiNoodle733:fix-drill-detail-page-size
Open

fix: Restrict DrillDetailTable pageSizeOptions to only valid option (50)#37733
RamiNoodle733 wants to merge 1 commit intoapache:masterfrom
RamiNoodle733:fix-drill-detail-page-size

Conversation

@RamiNoodle733
Copy link

Problem

The DrillDetailTable has a fixed page size of 50 (PAGE_SIZE constant), but the pageSizeOptions was not restricted, allowing users to select 5, 15, 25, 50, 100 from a dropdown. This caused confusion as selecting other values didn't work properly since the backend always returns 50 rows per page.

Solution

Added pageSizeOptions={[PAGE_SIZE]} to the Table component in DrillDetailPane.tsx to restrict the dropdown to only show the valid option (50).

Before

Users could select page sizes other than 50, but the table would still show 50 rows, causing a mismatch between the UI and actual data.

After

The page size dropdown only shows 50 as an option, which matches the actual behavior.

Fixes #36267

@bito-code-review
Copy link
Contributor

bito-code-review bot commented Feb 6, 2026

Code Review Agent Run #883552

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 42971b5..42971b5
    • superset-frontend/src/components/Chart/DrillDetail/DrillDetailPane.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 Logo

@dosubot dosubot bot added the change:frontend Requires changing the frontend label Feb 6, 2026
Fixes apache#36267

The pageSize was hardcoded to 50 but pageSizeOptions allowed selecting other values (5, 15, 25, 100), causing pagination issues. This change restricts pageSizeOptions to only ['50'] since that's the only valid page size supported by the component.
@RamiNoodle733 RamiNoodle733 force-pushed the fix-drill-detail-page-size branch from 42971b5 to 289b8fb Compare February 6, 2026 07:17
@bito-code-review
Copy link
Contributor

bito-code-review bot commented Feb 6, 2026

Code Review Agent Run #7ee860

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 289b8fb..289b8fb
    • superset-frontend/src/components/Chart/DrillDetail/DrillDetailPane.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 Logo

@sadpandajoe
Copy link
Member

I think if we want to only show 50 there needs to be a larger discussion as a whole. If you would want to fix it for the other options, then a discussion is probably not needed.

@sadpandajoe sadpandajoe added the hold! On hold label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:frontend Requires changing the frontend hold! On hold size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DrillDetailTable's pageSize is 50, but the pageSizeOptions allows the default 5,15,25,50,100

2 participants