Skip to content
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

Aggregations order : Escaping the user defined AGG_NAME #8042

Closed
aaneja opened this issue Oct 9, 2014 · 3 comments
Closed

Aggregations order : Escaping the user defined AGG_NAME #8042

aaneja opened this issue Oct 9, 2014 · 3 comments
Labels

Comments

@aaneja
Copy link

aaneja commented Oct 9, 2014

The order by path has the form defined as -
AGG_SEPARATOR := '>'
METRIC_SEPARATOR := '.'
AGG_NAME :=
METRIC := <the name of the metric (in case of multi-value metrics aggregation)>
PATH := <AGG_NAME>[<AGG_SEPARATOR><AGG_NAME>]*[<METRIC_SEPARATOR>]

Since AGG_NAME can contain dots (.) which is the METRIC_SEPARATOR, is it possible to somehow quote the AGG_NAME ?

A similar problem would arise if the multi-value metric aggregation name is a part of AGG_NAME

@aaneja
Copy link
Author

aaneja commented Oct 9, 2014

I took a quick look at src/main/java/org/elasticsearch/search/aggregations/support/OrderPath.java and it doesn't look like there's any code to escape/quote parts of the order path.

@clintongormley
Copy link

Related to #6736

@colings86
Copy link
Contributor

closing in favour of #9059

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants