Skip to content

[Backport 2.0] [BUG] opensearch crashes on closed client connection before search reply#3653

Merged
kartg merged 1 commit intoopensearch-project:2.0from
reta:issue.3557.2.0
Jun 22, 2022
Merged

[Backport 2.0] [BUG] opensearch crashes on closed client connection before search reply#3653
kartg merged 1 commit intoopensearch-project:2.0from
reta:issue.3557.2.0

Conversation

@reta
Copy link
Copy Markdown
Contributor

@reta reta commented Jun 22, 2022

Signed-off-by: Andriy Redko andriy.redko@aiven.io

Description

Backport 3dba46e from #3626

Issues Resolved

Closes #3557

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • 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.

…ply (opensearch-project#3626) (opensearch-project#3645)

* [BUG] opensearch crashes on closed client connection before search reply

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Addressing code review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit 3dba46e)

Co-authored-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta reta added the backport PRs or issues specific to backporting features or enhancments label Jun 22, 2022
@reta reta requested a review from a team as a code owner June 22, 2022 17:10
@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

✅   Gradle Check success 3a9e677
Log 6223

Reports 6223

@kartg kartg merged commit 2502198 into opensearch-project:2.0 Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport PRs or issues specific to backporting features or enhancments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants