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.

Sign up for GitHub

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

Jump to bottom

DRILL-8452: Library upgrades #2823

Merged
merged 5 commits into from
Aug 24, 2023
Merged

DRILL-8452: Library upgrades #2823

merged 5 commits into from
Aug 24, 2023

Conversation

jnturton
Copy link
Contributor

@jnturton jnturton commented Aug 16, 2023 •
edited
Loading

DRILL-8452: Library upgrades

  • aircompressor.version -> 0.25
  • antlr.version -> 4.13.0 4.9.3
  • asm.version -> 9.5
  • avro.version -> 1.11.2
  • commons.compress.version -> 1.23.0
  • commons.validator.version -> 1.7
  • hbase.version -> 2.5.5 (Hadoop 2 profile)
  • hbase.version -> 2.5.5-hadoop3
  • hikari.version -> 5.0.1
  • httpclient.version -> 4.5.14
  • httpdlog-parser.version -> 5.10.0
  • jersey.version -> 2.40
  • jetty -> 9.4.51.v20230217
  • jna.version -> 5.13.0
  • joda.version -> 2.12.5
  • libthrift.version -> 0.18.1
  • log4j.version -> 2.20.0
  • maven.version -> 3.9.4
  • metrics.version -> 4.2.19
  • protostuff.version -> 1.8.0
  • snakeyaml.version -> 2.1
  • surefire.version -> 3.1.2
  • testcontainers.version -> 1.18.3

Documentation

N/A

Testing

Existing unit tests.

Rebased on the unmerged #2821 to ensure testing with Hadoop 3.3.6.

@jnturton jnturton added dependencies backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there labels Aug 16, 2023
@jnturton jnturton self-assigned this Aug 16, 2023
@jnturton jnturton force-pushed the 8452-lib-upgrades branch 14 times, most recently from 7765b7c to ed71182 Compare August 21, 2023 14:16
@jnturton jnturton marked this pull request as ready for review August 22, 2023 03:40
@jnturton jnturton requested a review from cgivre August 22, 2023 03:41
@jnturton jnturton mentioned this pull request Aug 22, 2023
Merged
@jnturton
Copy link
Contributor Author

@vvysotskyi @luocooong @cgivre I believe this PR is ready for a review.

cgivre
cgivre approved these changes Aug 23, 2023
View reviewed changes
Copy link
Contributor

@cgivre cgivre left a comment

Choose a reason for hiding this comment

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

LGTM +1

@cgivre
Copy link
Contributor

cgivre commented Aug 23, 2023

I wonder if we could get rid of httpclient and replace it with okhttp3 which is also in use elsewhere. I don't think that is for this PR however.

@jnturton jnturton merged commit 61cfd90 into apache:master Aug 24, 2023
8 checks passed
@jnturton jnturton deleted the 8452-lib-upgrades branch August 24, 2023 03:39
cgivre pushed a commit to cgivre/drill that referenced this pull request Nov 2, 2023
@jnturton @cgivre
DRILL-8452: Library upgrades (apache#2823)
0ff99dd
jnturton added a commit that referenced this pull request Dec 31, 2023
@jnturton
DRILL-8452: Library upgrades (#2823)
ca9baff
jnturton added a commit to jnturton/drill that referenced this pull request Dec 31, 2023
@jnturton
DRILL-8452: Library upgrades (apache#2823)
4abecea
jnturton added a commit to jnturton/drill that referenced this pull request Jan 3, 2024
@jnturton
DRILL-8452: Library upgrades (apache#2823)
822801b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cgivre cgivre cgivre approved these changes

@vvysotskyi vvysotskyi Awaiting requested review from vvysotskyi

@luocooong luocooong Awaiting requested review from luocooong

Assignees

@jnturton jnturton

Labels
backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there dependencies
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jnturton @cgivre