Skip to content

fix(performance): Use spans dataset for EAP txn summary search bar#111299

Merged
mjq merged 1 commit intomasterfrom
mjq/txn-summary-filter-dataset
Mar 23, 2026
Merged

fix(performance): Use spans dataset for EAP txn summary search bar#111299
mjq merged 1 commit intomasterfrom
mjq/txn-summary-filter-dataset

Conversation

@mjq
Copy link
Member

@mjq mjq commented Mar 23, 2026

The EAPSummaryContentInner component was using TransactionSearchQueryBuilder, which provides transaction-oriented field definitions (e.g. geo.country_code). Since the EAP page queries the spans dataset, the search bar should use spans attributes (e.g. user.geo.country_code) for validation and autocompletion.

Replace with useSpanSearchQueryBuilderProps + TraceItemSearchQueryBuilder, matching the pattern used across Insights pages.

Fixes BROWSE-464.

…h bar

The EAPSummaryContentInner component was using TransactionSearchQueryBuilder,
which provides transaction-oriented field definitions (e.g., geo.country_code).
Since the EAP page queries the spans dataset, the search bar should use spans
attributes (e.g., user.geo.country_code) for validation and autocompletion.

Replace with useSpanSearchQueryBuilderProps + TraceItemSearchQueryBuilder,
matching the pattern used across Insights pages.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 23, 2026
@mjq mjq marked this pull request as ready for review March 23, 2026 15:43
@mjq mjq requested a review from a team as a code owner March 23, 2026 15:43
@linear-code
Copy link

linear-code bot commented Mar 23, 2026

@mjq mjq merged commit 495e26a into master Mar 23, 2026
73 checks passed
@mjq mjq deleted the mjq/txn-summary-filter-dataset branch March 23, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants