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.

Sign up for GitHub

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

Jump to bottom

[Transform] Add support for missing_bucket in group_by #42941

Closed
hendrikmuhs opened this issue Jun 6, 2019 · 1 comment · Fixed by #59591
Closed

[Transform] Add support for missing_bucket in group_by #42941

hendrikmuhs opened this issue Jun 6, 2019 · 1 comment · Fixed by #59591
Assignees
@hendrikmuhs
Labels
>feature :ml/Transform Transform

Comments

@hendrikmuhs
Copy link

By default documents without a value for a given group_by are ignored. Like composite aggs it would be nice to support missing_bucket in group_by.

Side effect: Progress reporting ignores missing values, progress calculation must be changed accordingly when implementing this feature.
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@hendrikmuhs hendrikmuhs mentioned this issue Jun 7, 2019
Merged
@hendrikmuhs hendrikmuhs changed the title [ML-DataFrame] Add support for missing_bucket in group_by [Transform] Add support for missing_bucket in group_by Oct 25, 2019
@hendrikmuhs hendrikmuhs added the help wanted adoptme label Feb 13, 2020
@hendrikmuhs hendrikmuhs self-assigned this May 29, 2020
@hendrikmuhs hendrikmuhs mentioned this issue Jul 15, 2020
Merged
@hendrikmuhs hendrikmuhs removed the help wanted adoptme label Jul 15, 2020
hendrikmuhs pushed a commit that referenced this issue Jul 29, 2020
[Transform] add support for missing bucket (#59591)
004388f 
add support for "missing_bucket" in group_by

fixes #42941
fixes #55102
@hendrikmuhs hendrikmuhs mentioned this issue Jul 29, 2020
Merged
hendrikmuhs pushed a commit that referenced this issue Jul 30, 2020
[7.x][Transform] add support for missing bucket (#59591) (#60390)
aaed6b5 
add support for "missing_bucket" in group_by

fixes #42941
fixes #55102
backport #59591
@Mpdreamz Mpdreamz mentioned this issue Nov 16, 2020
Closed
61 tasks
@stevejgordon stevejgordon mentioned this issue Dec 17, 2020
Closed
@szabosteve szabosteve mentioned this issue Sep 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hendrikmuhs hendrikmuhs

Labels
>feature :ml/Transform Transform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Transform] add support for missing bucket hendrikmuhs/elasticsearch
2 participants
@hendrikmuhs @elasticmachine