Skip to content

Add composite aggregation#6214

Open
fulmicoton-dd wants to merge 1 commit intomainfrom
composite-aggregations-rebase
Open

Add composite aggregation#6214
fulmicoton-dd wants to merge 1 commit intomainfrom
composite-aggregations-rebase

Conversation

@fulmicoton-dd
Copy link
Collaborator

Summary

  • Rebases Add composite aggregation #5957 by @rdettai-sk onto main with the latest tantivy revision (545169c0d8) which includes composite aggregation support upstream
  • Adds composite aggregation types to quickwit-query aggregation mappings
  • Fixes Send+Sync constraint in controlled_directory.rs for the updated tantivy API
  • Adds REST API test cases for composite aggregations

Test plan

  • cargo check passes
  • cargo nextest run -p quickwit-query — 139 tests pass
  • cargo nextest run -p quickwit-indexing — 165 tests pass
  • cargo nextest run -p quickwit-search — 159 tests pass
  • CI passes

🤖 Generated with Claude Code

@fulmicoton-dd fulmicoton-dd force-pushed the composite-aggregations-rebase branch from a9b38fb to 32a8e64 Compare March 23, 2026 19:23
@fulmicoton fulmicoton requested review from congx4 and rdettai-sk March 23, 2026 19:24
@fulmicoton
Copy link
Collaborator

@rdettai-sk Merging your PR in this. Claude made it in a different PR for the rebase. (I hardocded you as the author the author so you get the github stats and street cred.

(git commit --author="Remi Dettai <remi.dettai@sekoia.io>")

Rebase of #5957 by @rdettai-sk onto main, updated to use
the latest tantivy revision (545169c0d8) which includes
composite aggregation support upstream.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fulmicoton-dd fulmicoton-dd force-pushed the composite-aggregations-rebase branch from 32a8e64 to 2914a35 Compare March 23, 2026 19:43
@fulmicoton fulmicoton enabled auto-merge (squash) March 23, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants