Skip to content

Optimize sub aggregation using bulk collection lucene apis - #19737

Closed
jainankitk wants to merge 38 commits into
opensearch-project:feature/3.x-lucenefrom
jainankitk:agg-perf-lucene
Closed

Optimize sub aggregation using bulk collection lucene apis#19737
jainankitk wants to merge 38 commits into
opensearch-project:feature/3.x-lucenefrom
jainankitk:agg-perf-lucene

Conversation

@jainankitk

@jainankitk jainankitk commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Description

Followup to #19573 for leveraging bulk collection lucene apis. Currently has changes for couple of unmerged due to which the diff looks much bigger.

Related Issues

Related to #19384

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ing sub aggregation

Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
@jainankitk
jainankitk changed the base branch from main to feature/3.x-lucene October 23, 2025 23:11
@jainankitk
jainankitk marked this pull request as ready for review October 23, 2025 23:12
@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 09d5c71: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

asimmahmood1 and others added 13 commits October 27, 2025 11:00
Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
…ent idea oh how to seperately filter rewrite path vs regular path.

Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
Signed-off-by: Asim Mahmood <asim.seng@gmail.com>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 8403592: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Ankit Jain <jainankitk@apache.org>
@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 80079be: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for e90eaca: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for 17c6b7a: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@jainankitk

Copy link
Copy Markdown
Contributor Author

Closing PR as few things have changed since it was created. Will open another followup PR

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.

2 participants