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

Throw UnsupportedOperationException in unused methods #15446

Merged
merged 1 commit into from
Aug 27, 2024

Conversation

andrross
Copy link
Member

These methods infinitely recurse as currently implemented. This change makes them throw UnsupportedOperationException similar to many other methods in this class.

Related Issues

Resolves #15387

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.

These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Copy link
Contributor

✅ Gradle check result for 8e70be1: SUCCESS

Copy link

codecov bot commented Aug 27, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 71.88%. Comparing base (46a7bb6) to head (8e70be1).
Report is 1 commits behind head on main.

Files Patch % Lines
...rg/opensearch/index/engine/TranslogLeafReader.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main   #15446   +/-   ##
=========================================
  Coverage     71.87%   71.88%           
- Complexity    63263    63283   +20     
=========================================
  Files          5225     5225           
  Lines        296214   296214           
  Branches      42788    42788           
=========================================
+ Hits         212914   212921    +7     
+ Misses        65815    65780   -35     
- Partials      17485    17513   +28     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reta reta merged commit 20ebe6e into opensearch-project:main Aug 27, 2024
60 of 62 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 27, 2024
These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit 20ebe6e)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@dblock
Copy link
Member

dblock commented Aug 28, 2024

Thanks for taking care of this @andrross.

reta pushed a commit that referenced this pull request Aug 28, 2024
These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.


(cherry picked from commit 20ebe6e)

Signed-off-by: Andrew Ross <andrross@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
akolarkunnu pushed a commit to akolarkunnu/OpenSearch that referenced this pull request Sep 10, 2024
…ject#15446)

These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 16, 2024
…ject#15446)

These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 17, 2024
…ject#15446)

These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 21, 2024
…ject#15446)

These methods infinitely recurse as currently implemented. This change
makes them throw UnsupportedOperationException similar to many other
methods in this class.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch bug Something isn't working Indexing Indexing, Bulk Indexing and anything related to indexing skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Couple of cases of infinite recursion
3 participants