search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
38.64k stars 4.14k forks source link

Equations using aggregates that have arguments with spaces need quotations #78064

Open narsaynorath opened 1 hour ago

narsaynorath commented 1 hour ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Go to discover
  2. Select an aggregate, e.g. count()
  3. Select a count_if aggregate and pass a value with spaces in it, e.g. count_if(title,equals,test transaction)
  4. Create an equation using the autocomplete to get count_if(title,equals,test transaction) / count()
  5. Apply the columns

Expected Result

We can see the results of the equation

Actual Result

There is an error saying "Unable to parse your equation, make sure it is well formed arithmetic"

Note: The current workaround is to manually apply quotes around the value with spaces, i.e. count_if(title,equals,"test transaction") in both the aggregate and the equation

Product Area

Discover

Link

https://sentry.sentry.io/discover/homepage/?dataset=transactions&field=count%28%29&field=count_if%28title%2Cequals%2Ctest%20transaction%29&field=equation%7Ccount_if%28title%2Cequals%2Ctest%20transaction%29%20%2F%20count%28%29&id=29115&name=&project=1&query=&queryDataset=transaction-like&sort=-count%28%29&statsPeriod=24h&topEvents=5&yAxis=count%28%29

DSN

No response

Version

No response

getsantry[bot] commented 1 hour ago

Routing to @getsentry/product-owners-discover for triage ⏲️