Skip to content

fix(search): Parse backtick-delimited aggregate query arguments - #123897

Open
vgrozdanic wants to merge 1 commit into
masterfrom
fix/search-query-aggregate-arguments
Open

fix(search): Parse backtick-delimited aggregate query arguments#123897
vgrozdanic wants to merge 1 commit into
masterfrom
fix/search-query-aggregate-arguments

Conversation

@vgrozdanic

Copy link
Copy Markdown
Member

Support backtick-delimited search predicates as a single argument to conditional aggregates, including Boolean groups, quoted values, and lists:

count_if(`span.description:"hello world" AND (x:1 OR y:2)`,span.duration):>0

Today, the parser does not recognize backtick-delimited arguments. Spaces, parentheses, quotes, and commas inside the predicate can break argument parsing and turn the aggregate into unrelated search terms.

Preserve grouped Boolean predicates and quoted values inside backtick-delimited arguments.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 9, 2026
@vgrozdanic
vgrozdanic marked this pull request as ready for review September 9, 2026 12:54
@vgrozdanic
vgrozdanic requested a review from a team as a code owner September 9, 2026 12:54
@vgrozdanic
vgrozdanic added this pull request to stack #123900 September 9, 2026 13:33
@vgrozdanic
vgrozdanic removed this pull request from stack #123900 September 9, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant