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

[Backport 2.x] Update snappy to 1.1.10.1 and guava to 32.0.1-jre (#2886) #2889

Merged
merged 2 commits into from
Jun 22, 2023

Conversation

cwperks
Copy link
Member

@cwperks cwperks commented Jun 21, 2023

Backport #2886 to 2.x

…#2886)

* Update snappy to 1.1.10.1 and guava to 32.0.1-jre

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Upgrade kafka to 3.5.0

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Force snappy

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Add runtime dependency on org.scala-lang.modules:scala-java8-compat_3:1.0.2 to fix issue with KafkaSinkTest

Signed-off-by: Craig Perkins <cwperx@amazon.com>

---------

Signed-off-by: Craig Perkins <cwperx@amazon.com>
(cherry picked from commit d871af3)
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #2889 (29940e2) into 2.x (2426102) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.x    #2889      +/-   ##
============================================
- Coverage     61.49%   61.43%   -0.06%     
+ Complexity     3381     3377       -4     
============================================
  Files           264      264              
  Lines         18645    18645              
  Branches       3287     3287              
============================================
- Hits          11465    11455      -10     
- Misses         5597     5603       +6     
- Partials       1583     1587       +4     

see 4 files with indirect coverage changes

@cwperks cwperks merged commit 3e3b776 into opensearch-project:2.x Jun 22, 2023
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