Skip to content

[7.x] Allow creating filters from fields with null values in discover (#70936)#71211

Merged
lizozom merged 5 commits intoelastic:7.xfrom
lizozom:backport/7.x/pr-70936
Jul 13, 2020
Merged

[7.x] Allow creating filters from fields with null values in discover (#70936)#71211
lizozom merged 5 commits intoelastic:7.xfrom
lizozom:backport/7.x/pr-70936

Conversation

@lizozom
Copy link
Copy Markdown
Contributor

@lizozom lizozom commented Jul 9, 2020

Backports the following commits to 7.x:

…tic#70936)

* Fix bug elastic#7189

* typo

* Test adjustments

* wait for load complete

* Fine tune test

* Update src/plugins/data/public/query/filter_manager/lib/generate_filters.ts

Co-authored-by: Lukas Olson <olson.lukas@gmail.com>

* Fix filtering by an array of nulls
Allow filtering by a non existing field in the doc
simplify flatten hit logic

Co-authored-by: Lukas Olson <olson.lukas@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@lizozom lizozom added the backport This PR is a backport of another PR label Jul 9, 2020
@lizozom
Copy link
Copy Markdown
Contributor Author

lizozom commented Jul 9, 2020

@elasticmachine merge upstream

@lizozom
Copy link
Copy Markdown
Contributor Author

lizozom commented Jul 13, 2020

@elasticmachine merge upstream

1 similar comment
@lizozom
Copy link
Copy Markdown
Contributor Author

lizozom commented Jul 13, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@lizozom lizozom merged commit aaf0abe into elastic:7.x Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants