Skip to content

docs: update workbook default query row limit to 10,000#11003

Merged
keydunov merged 1 commit into
masterfrom
docs/workbook-default-row-limit-10k
Jun 3, 2026
Merged

docs: update workbook default query row limit to 10,000#11003
keydunov merged 1 commit into
masterfrom
docs/workbook-default-row-limit-10k

Conversation

@keydunov
Copy link
Copy Markdown
Member

@keydunov keydunov commented Jun 3, 2026

What changed

Updates the documented default row limit for workbook semantic queries from 5,000 to 10,000 in docs/content/product/exploration/workbooks/querying-data.mdx.

Why

The default row limit for workbook queries is 10,000; the docs lagged at 5,000. This brings the documentation in line with the current product default.

Notes for reviewers

  • Only the default value changed. The adjustable maximum (50,000) on the same line is unchanged.
  • This is the workbook UI limit, distinct from the Core Data API default (CUBEJS_DB_QUERY_DEFAULT_LIMIT, documented separately as 10,000).

🤖 Generated with Claude Code

The default row limit for workbook semantic queries was documented as
5,000; update it to 10,000 to match the current product default. The
adjustable maximum (50,000) is unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jun 3, 2026

Claude finished @keydunov's task in 18s —— View job


Code Review

Verdict: LGTM ✅ — 0 high, 0 medium, 0 low

Review details
  • Read PR diff
  • Verify change matches PR description
  • Check for related stale docs

Change reviewed: docs/content/product/exploration/workbooks/querying-data.mdx — single-line update of the documented workbook default row limit from 5,000 → 10,000.

Findings:

  • Code Quality: Minimal, accurate, scoped to the one stale value. The adjustable max (50,000) is correctly left intact.
  • Security / Performance / Testing: N/A — docs-only change.
  • Documentation accuracy: PR description correctly distinguishes this workbook UI limit from CUBEJS_DB_QUERY_DEFAULT_LIMIT (Core Data API), which is documented separately. No conflicting references found.

Nothing to fix — safe to merge.

• Branch: [`docs/workbook-default-row-limit-10k`](https://github.com/cube-js/cube/tree/docs/workbook-default-row-limit-10k)

@keydunov keydunov merged commit 9576afe into master Jun 3, 2026
14 checks passed
@keydunov keydunov deleted the docs/workbook-default-row-limit-10k branch June 3, 2026 03:28
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