pt/dataflow
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Pivot: skip inspector query when no group_by hierarchy is active

Browse Source 
Without group_by there are no coordinate filters, so the view query
would return the full dataset and hang. Early-return on click if
config.group_by is empty.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Paul Trowbridge 2026-04-15 08:52:09 -04:00
parent 6e9cdd82ea
commit 8d3cc24094
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ui/src/pages/Pivot.jsx
View File

@ -150,6 +150,12 @@ export default function Pivot({ source }) {
if (!row) return if (!row) return
const eventFilters = (detail.config || {}).filter || [] const eventFilters = (detail.config || {}).filter || []
const config = await viewer.save() const config = await viewer.save()
// Without a group_by hierarchy there are no coordinate filters, so the
// query would return the entire dataset — skip the inspector in that case
const hasHierarchy = (config.group_by || []).length > 0
if (!hasHierarchy) return
setClickDetail({ row, config, column_names, eventFilters }) setClickDetail({ row, config, column_names, eventFilters })
// Use a Perspective view with the event filters + expressions so computed // Use a Perspective view with the event filters + expressions so computed