Skip to content

Conversation

@vigyasharma
Copy link
Contributor

Description

Update testing.md to include documentation about debug-server-jvm, a gradle task that starts a debug server within opensearch allowing you to connect your IDE debugger clients.

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.

@vigyasharma vigyasharma requested a review from a team as a code owner October 8, 2025 19:20
Copy link
Member

@owaiskazi19 owaiskazi19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @vigyasharma

@owaiskazi19 owaiskazi19 merged commit 13b1bc2 into opensearch-project:main Oct 8, 2025
35 of 36 checks passed
@reta
Copy link
Contributor

reta commented Oct 8, 2025

Thanks @vigyasharma!

aparajita31pandey pushed a commit to aparajita31pandey/OpenSearch that referenced this pull request Oct 9, 2025
rgsriram pushed a commit to rgsriram/OpenSearch that referenced this pull request Oct 11, 2025
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
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants