Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Transport Request #163

Merged
merged 1 commit into from
Oct 13, 2022
Merged

Conversation

VijayanB
Copy link
Member

Description

Remove usage of deprecated BaseNodeRequest

Signed-off-by: Vijayan Balasubramanian balasvij@amazon.com

Check List

  • Commits are signed per the DCO using --signoff

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.

Remove usage of deprecated BaseNodeRequest

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
@VijayanB VijayanB requested a review from a team October 11, 2022 22:43
@codecov-commenter
Copy link

Codecov Report

Merging #163 (fdcae01) into main (1eebc3f) will increase coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #163      +/-   ##
============================================
+ Coverage     84.85%   85.13%   +0.28%     
- Complexity      319      320       +1     
============================================
  Files            44       44              
  Lines          1063     1063              
  Branches         86       86              
============================================
+ Hits            902      905       +3     
+ Misses          126      124       -2     
+ Partials         35       34       -1     
Impacted Files Coverage Δ
...eospatial/stats/upload/UploadStatsNodeRequest.java 0.00% <ø> (ø)
...earch/geospatial/index/mapper/xypoint/XYPoint.java 74.13% <0.00%> (+5.17%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@navneet1v navneet1v left a comment

Choose a reason for hiding this comment

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

Add the issue which points that BaseNodeRequest is deprecated.

@VijayanB
Copy link
Member Author

As part of this OpenSearch issue opensearch-project/OpenSearch#2747 BaseNodeRequest class is removed in 3.0 and deprecated in 2.x . Use TransportRequest which was super class for BaseNodeRequest as an alternative.

@VijayanB VijayanB requested review from navneet1v and a team October 12, 2022 22:14
@heemin32 heemin32 merged commit 8269730 into opensearch-project:main Oct 13, 2022
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.

5 participants