Add second level of field collapsing#31808
Conversation
|
Pinging @elastic/es-search-aggs |
jimczi
left a comment
There was a problem hiding this comment.
It looks good @mayya-sharipova but I wonder if the second collapse should be under inner_hits ? I know we discussed the proposed format already but I forgot that we can have multiple inner_hits in a single collapse so it would be more flexible and consistent to define the collapse field there ?
|
@jimczi Thanks Jim, I will try to implement |
|
@jimczi I have modified the PR to make the second level collapse under |
jimczi
left a comment
There was a problem hiding this comment.
Thanks @mayya-sharipova, it looks great !
Put second level collapse under inner_hits Closes #24855
* 6.x: Watcher: Make settings reloadable (#31746) [Rollup] Histo group config should support scaled_floats (#32048) lazy snapshot repository initialization (#31606) Add secure setting for watcher email password (#31620) Watcher: cleanup ensureWatchExists use (#31926) Add second level of field collapsing (#31808) Added lenient flag for synonym token filter (#31484) (#31970) Test: Fix a second case of bad watch creation [Rollup] Use composite's missing_bucket (#31402) Docs: Restyled cloud link in getting started Docs: Change formatting of Cloud options Re-instate link in StringFunctionUtils javadocs Correct spelling of AnalysisPlugin#requriesAnalysisSettings (#32025) Fix problematic chars in javadoc [ML] Move open job failure explanation out of root cause (#31925) [ML] Switch ML native QA tests to use a 3 node cluster (#32011)
as per elastic/elasticsearch#31808 (cherry picked from commit 7d78767)
Closes #24855