-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rollup: split series in a visualization + sorting by average creates an invalid query #34782
Comments
Hi, thanks a lot for the detailed report. Does that same chart configuration work on a regular (non rolled up) index work? That error sounds like it would intepret the order path now as a path not a unique name, which sounds a bit strange to me, that this error should only happen in the rollup API? Could you please verify that the same visualization on the non rolledup index doesn't throw that error? Cheers, |
Pinging @elastic/kibana-app |
Hi @timroes ! It did not fail when running it directly against the indices instead of the rolled up index pattern. I just added both visualizations to a dashboard to inspect them; the working one is sending:
and the non working one (the rollup one) is sending:
The only difference that I see is that the normal one calls to Checking further, I think it may be related to elastic/elasticsearch#30467 , not to kibana. |
Yes I am pretty sure elastic/elasticsearch#30467 is the actual issue here. I would close this, and refer to the ES issue for further tracking the fix of that. |
Kibana version: 6.7.0
Elasticsearch version: 6.7.0
Server OS version: Windows 10
Browser version: 73.0.3683.86 (Official Build) (64-bit)
Browser OS version: Windows 10
Original install method (e.g. download page, yum, from source, etc.): Download
Describe the bug:
When having a rollup index with an average aggregation, if we split the aggregation by some term and sort by that average value, it fails.
Steps to reproduce:
Vertical bar
,y-axis: the average
,x-axis: timestamp
,Split the series by the term
,Sort by average descending
At this point, there is an error:
Expected behavior:
The visualization does not fail
Screenshots (if relevant):
Errors in browser console (if relevant):
Provide logs and/or server output (if relevant):
The most relevant error is in Elasticsearch logs:
Expand error
Any additional context:
This seems related to using dots in the aggregation names. Copy-pasting the search from the logs, I can reproduce the same failure via dev tools - note that the aggregation is called
1.value
:Non working aggregation
If I rename the aggregation from
1.value
tonodots
- and update the reference to the aggregation in the sort - it works fine:Working aggregation
The text was updated successfully, but these errors were encountered: