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

[Transform] cannot group by boolean #58697

Closed
hendrikmuhs opened this issue Jun 29, 2020 · 1 comment · Fixed by #58741
Closed

[Transform] cannot group by boolean #58697

hendrikmuhs opened this issue Jun 29, 2020 · 1 comment · Fixed by #58741
Assignees
Labels

Comments

@hendrikmuhs
Copy link
Contributor

Reported via discuss:

Grouping boolean using a terms aggregation throws in transform:

{
  "error" : {
    "root_cause" : [
      {
        "type" : "illegal_argument_exception",
        "reason" : "Value of type [class java.lang.Boolean] is not supported"
      }
    ],
    "type" : "illegal_argument_exception",
    "reason" : "Value of type [class java.lang.Boolean] is not supported"
  },
  "status" : 400
}

The above is possible with aggs and composite aggs. The error message originates from IDGenerator which does not know how to treat a boolean.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@benwtrent benwtrent self-assigned this Jun 30, 2020
benwtrent added a commit that referenced this issue Jun 30, 2020
Since the underlying composite aggs support boolean mapped values for terms, transforms should also support them

closes #58697
benwtrent added a commit to benwtrent/elasticsearch that referenced this issue Jun 30, 2020
)

Since the underlying composite aggs support boolean mapped values for terms, transforms should also support them

closes elastic#58697
benwtrent added a commit that referenced this issue Jun 30, 2020
…58760)

Since the underlying composite aggs support boolean mapped values for terms, transforms should also support them

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

Successfully merging a pull request may close this issue.

3 participants