Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport b4413a7 from #2876

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit b4413a7)
@cwperks
Copy link
Member

cwperks commented Jun 20, 2023

@peternied The backports needs to be modified to account for Apache HttpClient4 in the 2.x line instead of Apache HttpClient5 which is in main.

@codecov
Copy link

codecov bot commented Jun 20, 2023

Codecov Report

Merging #2879 (2426102) into 2.x (2426102) will not change coverage.
The diff coverage is n/a.

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

@@            Coverage Diff            @@
##                2.x    #2879   +/-   ##
=========================================
  Coverage     61.49%   61.49%           
  Complexity     3381     3381           
=========================================
  Files           264      264           
  Lines         18645    18645           
  Branches       3287     3287           
=========================================
  Hits          11465    11465           
  Misses         5597     5597           
  Partials       1583     1583           

@cwperks
Copy link
Member

cwperks commented Jun 21, 2023

@mingshl These backports need to be updated to account for Apache HttpClient4 on 2.x instead of HttpClient5 on main. I can help update the backports tomorrow if needed.

@cwperks
Copy link
Member

cwperks commented Jun 22, 2023

Closing in favor of: #2893

@cwperks cwperks closed this Jun 22, 2023
@peternied peternied deleted the backport/backport-2876-to-2.x branch July 10, 2023 20:47
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.

3 participants