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

In terms aggs format cannot be used with "value_type": "string" #68077

Open
imotov opened this issue Jan 27, 2021 · 4 comments
Open

In terms aggs format cannot be used with "value_type": "string" #68077

imotov opened this issue Jan 27, 2021 · 4 comments
Labels
:Analytics/Aggregations Aggregations >bug stalled Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@imotov
Copy link
Contributor

imotov commented Jan 27, 2021

Specifying format in terms aggregation with "value_type": "string" causes unhelpful unsupported_operation_exception. I didn't verify, but based on the source of the exception the issue most likely exist in all other value sources.

To reproduce:

DELETE test?ignore_unavailable

POST test/_bulk?refresh
{"index": {}}
{"foo": 3}


POST test/_search?error_trace
{
  "size": 0,
  "aggs": {
    "test": {
      "terms": {
        "value_type": "string",
        "field": "foo",
        "format": "$#.00",
        "size": 10
      }
    }
  }
}
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jan 27, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@imotov imotov self-assigned this Jan 27, 2021
@imotov imotov added the stalled label Mar 9, 2021
@imotov
Copy link
Contributor Author

imotov commented Mar 9, 2021

We need to make sure our format is consistent first, so I will address this after #47469.

@wchaparro
Copy link
Member

Hey @imotov should we change the exception message as a first step, and then next step would require #47469 ?
WDYT?

@imotov
Copy link
Contributor Author

imotov commented Sep 15, 2021

WDYT?

I still think it would be much cleaner to address after #47469.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations >bug stalled Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Development

No branches or pull requests

3 participants