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

[Extensions] Changing ExtensionActionRequest streaminput constructor to be public #6101

Merged
merged 3 commits into from
Jan 31, 2023

Conversation

joshpalis
Copy link
Member

Signed-off-by: Joshua Palis jpalis@amazon.com

Description

Manual backport PR to 2.x due to backport failure here : #6094 (comment)

Check List

  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

Signed-off-by: Joshua Palis <jpalis@amazon.com>
Signed-off-by: Joshua Palis <jpalis@amazon.com>
CHANGELOG.md Outdated
@@ -12,6 +12,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
- Replace latches with CompletableFutures for extensions ([#5646](https://github.com/opensearch-project/OpenSearch/pull/5646))
- Add support to disallow search request with preference parameter with strict weighted shard routing([#5874](https://github.com/opensearch-project/OpenSearch/pull/5874))
- Added support to apply index create block ([#4603](https://github.com/opensearch-project/OpenSearch/issues/4603))
- Changing ExtensionActionRequest streaminput constructor to be public ([#6101](https://github.com/opensearch-project/OpenSearch/pull/6101))
Copy link
Member

Choose a reason for hiding this comment

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

See #6094 (comment), but I don't think we need a changelog entry for these types of changes.

Copy link
Member Author

Choose a reason for hiding this comment

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

ill go ahead and remove this

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.indices.replication.SegmentReplicationRelocationIT.testPrimaryRelocationWithSegRepFailure

@codecov-commenter
Copy link

Codecov Report

Merging #6101 (b8db233) into 2.x (457daa2) will increase coverage by 0.05%.
The diff coverage is n/a.

❗ Current head b8db233 differs from pull request most recent head a8bcd2d. Consider uploading reports for the commit a8bcd2d to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #6101      +/-   ##
============================================
+ Coverage     70.42%   70.48%   +0.05%     
+ Complexity    59007    58965      -42     
============================================
  Files          4771     4771              
  Lines        282733   282733              
  Branches      41179    41179              
============================================
+ Hits         199121   199277     +156     
+ Misses        66994    66752     -242     
- Partials      16618    16704      +86     
Impacted Files Coverage Δ
...arch/extensions/action/ExtensionActionRequest.java 100.00% <ø> (ø)
...n/indices/forcemerge/ForceMergeRequestBuilder.java 0.00% <0.00%> (-75.00%) ⬇️
.../indices/forcemerge/TransportForceMergeAction.java 25.00% <0.00%> (-58.34%) ⬇️
...a/org/opensearch/client/cluster/SniffModeInfo.java 0.00% <0.00%> (-56.25%) ⬇️
...nsearch/rest/action/cat/RestCatRecoveryAction.java 43.61% <0.00%> (-42.56%) ⬇️
...h/action/ingest/SimulateDocumentVerboseResult.java 60.71% <0.00%> (-39.29%) ⬇️
...earch/client/indices/GetIndexTemplatesRequest.java 46.15% <0.00%> (-38.47%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
...arch/search/aggregations/pipeline/LinearModel.java 23.07% <0.00%> (-34.62%) ⬇️
...rg/opensearch/index/shard/ShardSplittingQuery.java 34.35% <0.00%> (-32.83%) ⬇️
... and 509 more

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

Signed-off-by: Joshua Palis <jpalis@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@ryanbogan ryanbogan merged commit fc678a8 into opensearch-project:2.x Jan 31, 2023
@joshpalis joshpalis deleted the extensionaction branch January 31, 2023 01:24
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.

5 participants