Adds a mapper for context aware segments grouping criteria #19233
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
❌ Gradle check result for 4009145: 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? |
shatejas
force-pushed
the
namespace-mapper
branch
3 times, most recently
from
3298285 to
1147e4a
Compare
shatejas
changed the title
shatejas
requested review from
a team,
Bukhtawar,
CEHENKLE,
Rishikesh1159,
anasalkouz,
andrross,
ashking94,
cwperks,
dbwiddis,
gbbafna,
jed326,
kotwanikunal,
mch2,
msfroh,
owaiskazi19,
reta,
sachinpkale,
saratvemulapalli,
shwetathareja and
sohami
as code owners
shatejas
force-pushed
the
namespace-mapper
branch
from
f9a9828 to
94b7ce4
Compare
Contributor
|
❌ Gradle check result for ddb577b: 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? |
shatejas
force-pushed
the
namespace-mapper
branch
from
ddb577b to
1960464
Compare
The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]>
Signed-off-by: Tejas Shah <[email protected]>
…teg tests Signed-off-by: Tejas Shah <[email protected]>
shatejas
force-pushed
the
namespace-mapper
branch
from
507dcd2 to
d5f920b
Compare
Contributor
|
❌ Gradle check result for ec031b2: 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? |
Signed-off-by: Tejas Shah <[email protected]>
shatejas
force-pushed
the
namespace-mapper
branch
from
ec031b2 to
3be7f35
Compare
Contributor
|
❌ Gradle check result for 3be7f35: 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? |
Contributor
|
❌ Gradle check result for 81c6291: 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? |
Signed-off-by: Tejas Shah <[email protected]>
shatejas
force-pushed
the
namespace-mapper
branch
from
81c6291 to
5a9e26c
Compare
Contributor
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #19233 +/- ##
============================================
+ Coverage 73.00% 73.04% +0.04%
- Complexity 70534 70585 +51
============================================
Files 5719 5722 +3
Lines 323260 323387 +127
Branches 46816 46833 +17
============================================
+ Hits 235993 236220 +227
+ Misses 68224 68063 -161
- Partials 19043 19104 +61 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This was referenced Oct 9, 2025
aparajita31pandey
pushed a commit
to aparajita31pandey/OpenSearch
that referenced
this pull request
Oct 9, 2025
…h-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]>
rgsriram
pushed a commit
to rgsriram/OpenSearch
that referenced
this pull request
Oct 11, 2025
…h-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]>
rgsriram
pushed a commit
to rgsriram/OpenSearch
that referenced
this pull request
Oct 11, 2025
…h-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]>
karenyrx
pushed a commit
that referenced
this pull request
Oct 14, 2025
* Introduce Interceptor to Grpc Plugin - Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * This is a combination of 7 commits. Add testcases for gRPC Interceptor Signed-off-by: Aparajita Pandey <[email protected]> Ran spotlessAppy Signed-off-by: Aparajita Pandey <[email protected]> refactor Signed-off-by: Aparajita Pandey <[email protected]> spotlesssCheck Signed-off-by: Aparajita Pandey <[email protected]> check if it verified Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain check if it verified Signed-off-by: Aparajita Pandey <[email protected]> revert Signed-off-by: Aparajita Pandey <[email protected]> empty commit Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> testCases Signed-off-by: Aparajita Pandey <[email protected]> logger Signed-off-by: Aparajita Pandey <[email protected]> Resolve Conflict Signed-off-by: Aparajita Pandey <[email protected]> javadoc Signed-off-by: Aparajita Pandey <[email protected]> * javadoc Signed-off-by: Aparajita Pandey <[email protected]> * cleanuo Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * build.gradle fixes Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * rebase Signed-off-by: Aparajita Pandey <[email protected]> * coverage Signed-off-by: Aparajita Pandey <[email protected]> * spotless Signed-off-by: Aparajita Pandey <[email protected]> * Add logic to throw exception on workload group deletion with associated rules (#19502) Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * add documentation about debug-server-jvm in testing.md (#19567) Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Adds a mapper for context aware segments grouping criteria (#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Refactor README.md and GrpcInterceptorChain Constructor Signed-off-by: Aparajita Pandey <[email protected]> * UpdateREADME & switch to implmentation Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * Address PR review comments - Changed SPI dependencies from api to implementation - Made OrderedGrpcInterceptor a nested interface inside GrpcInterceptorProvider - Updated README with build.gradle extension declaration examples - Simplified duplicate order handling documentation Signed-off-by: Aparajita Pandey <[email protected]> * empty commit Signed-off-by: Aparajita Pandey <[email protected]> * resolve conflict Signed-off-by: Aparajita Pandey <[email protected]> --------- Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> Co-authored-by: Kaushal Kumar <[email protected]> Co-authored-by: Vigya Sharma <[email protected]> Co-authored-by: Tejas Shah <[email protected]>
peteralfonsi
pushed a commit
to peteralfonsi/OpenSearch
that referenced
this pull request
Oct 15, 2025
…h-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]>
peteralfonsi
pushed a commit
to peteralfonsi/OpenSearch
that referenced
this pull request
Oct 15, 2025
…ct#19005) * Introduce Interceptor to Grpc Plugin - Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * This is a combination of 7 commits. Add testcases for gRPC Interceptor Signed-off-by: Aparajita Pandey <[email protected]> Ran spotlessAppy Signed-off-by: Aparajita Pandey <[email protected]> refactor Signed-off-by: Aparajita Pandey <[email protected]> spotlesssCheck Signed-off-by: Aparajita Pandey <[email protected]> check if it verified Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain check if it verified Signed-off-by: Aparajita Pandey <[email protected]> revert Signed-off-by: Aparajita Pandey <[email protected]> empty commit Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> testCases Signed-off-by: Aparajita Pandey <[email protected]> logger Signed-off-by: Aparajita Pandey <[email protected]> Resolve Conflict Signed-off-by: Aparajita Pandey <[email protected]> javadoc Signed-off-by: Aparajita Pandey <[email protected]> * javadoc Signed-off-by: Aparajita Pandey <[email protected]> * cleanuo Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * build.gradle fixes Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * rebase Signed-off-by: Aparajita Pandey <[email protected]> * coverage Signed-off-by: Aparajita Pandey <[email protected]> * spotless Signed-off-by: Aparajita Pandey <[email protected]> * Add logic to throw exception on workload group deletion with associated rules (opensearch-project#19502) Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * add documentation about debug-server-jvm in testing.md (opensearch-project#19567) Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Adds a mapper for context aware segments grouping criteria (opensearch-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Refactor README.md and GrpcInterceptorChain Constructor Signed-off-by: Aparajita Pandey <[email protected]> * UpdateREADME & switch to implmentation Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * Address PR review comments - Changed SPI dependencies from api to implementation - Made OrderedGrpcInterceptor a nested interface inside GrpcInterceptorProvider - Updated README with build.gradle extension declaration examples - Simplified duplicate order handling documentation Signed-off-by: Aparajita Pandey <[email protected]> * empty commit Signed-off-by: Aparajita Pandey <[email protected]> * resolve conflict Signed-off-by: Aparajita Pandey <[email protected]> --------- Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> Co-authored-by: Kaushal Kumar <[email protected]> Co-authored-by: Vigya Sharma <[email protected]> Co-authored-by: Tejas Shah <[email protected]>
rgsriram
pushed a commit
to rgsriram/OpenSearch
that referenced
this pull request
Oct 18, 2025
…ct#19005) * Introduce Interceptor to Grpc Plugin - Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * This is a combination of 7 commits. Add testcases for gRPC Interceptor Signed-off-by: Aparajita Pandey <[email protected]> Ran spotlessAppy Signed-off-by: Aparajita Pandey <[email protected]> refactor Signed-off-by: Aparajita Pandey <[email protected]> spotlesssCheck Signed-off-by: Aparajita Pandey <[email protected]> check if it verified Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain check if it verified Signed-off-by: Aparajita Pandey <[email protected]> revert Signed-off-by: Aparajita Pandey <[email protected]> empty commit Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> Added GrpcInterceptorChain Signed-off-by: Aparajita Pandey <[email protected]> testCases Signed-off-by: Aparajita Pandey <[email protected]> logger Signed-off-by: Aparajita Pandey <[email protected]> Resolve Conflict Signed-off-by: Aparajita Pandey <[email protected]> javadoc Signed-off-by: Aparajita Pandey <[email protected]> * javadoc Signed-off-by: Aparajita Pandey <[email protected]> * cleanuo Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * move GrpcInterceptor Interface to spi Signed-off-by: Aparajita Pandey <[email protected]> * build.gradle fixes Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * GrpcInterceptorChain refactoring Signed-off-by: Aparajita Pandey <[email protected]> * rebase Signed-off-by: Aparajita Pandey <[email protected]> * coverage Signed-off-by: Aparajita Pandey <[email protected]> * spotless Signed-off-by: Aparajita Pandey <[email protected]> * Add logic to throw exception on workload group deletion with associated rules (opensearch-project#19502) Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * add documentation about debug-server-jvm in testing.md (opensearch-project#19567) Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Adds a mapper for context aware segments grouping criteria (opensearch-project#19233) * Adds Context aware grouping mapper The change also stores grouping criteria for each doc in parsed context Signed-off-by: Tejas Shah <[email protected]> * Adds a test for ContextAwareGroupingScript Signed-off-by: Tejas Shah <[email protected]> * Adds ContextAwareGroupingScript support in mockScriptEngine to fix integ tests Signed-off-by: Tejas Shah <[email protected]> * Corrects CHANGELOG.md Signed-off-by: Tejas Shah <[email protected]> * Fixes LangPainlessClientYamlTestSuiteIT Signed-off-by: Tejas Shah <[email protected]> --------- Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> * Refactor README.md and GrpcInterceptorChain Constructor Signed-off-by: Aparajita Pandey <[email protected]> * UpdateREADME & switch to implmentation Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * refactoring Signed-off-by: Aparajita Pandey <[email protected]> * Address PR review comments - Changed SPI dependencies from api to implementation - Made OrderedGrpcInterceptor a nested interface inside GrpcInterceptorProvider - Updated README with build.gradle extension declaration examples - Simplified duplicate order handling documentation Signed-off-by: Aparajita Pandey <[email protected]> * empty commit Signed-off-by: Aparajita Pandey <[email protected]> * resolve conflict Signed-off-by: Aparajita Pandey <[email protected]> --------- Signed-off-by: Aparajita Pandey <[email protected]> Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Vigya Sharma <[email protected]> Signed-off-by: Tejas Shah <[email protected]> Signed-off-by: Aparajita Pandey <[email protected]> Co-authored-by: Kaushal Kumar <[email protected]> Co-authored-by: Vigya Sharma <[email protected]> Co-authored-by: Tejas Shah <[email protected]>
This was referenced Dec 5, 2025
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This is a new mapper for context aware segments. It also has indexing side of changes and stores the grouping criteria in parsed documents
Related Issues
RFC: #19223
Check List
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.