Skip to content

Update Drift version to 1.36 (second attempt)#19953

Merged
ajaygeorge merged 1 commit intoprestodb:masterfrom
sotodel:master
Jun 26, 2023
Merged

Update Drift version to 1.36 (second attempt)#19953
ajaygeorge merged 1 commit intoprestodb:masterfrom
sotodel:master

Conversation

@sotodel
Copy link
Contributor

@sotodel sotodel commented Jun 22, 2023

The Drift update was reverted while we figured out some NoClassDefFoundErrors. Those have been resolved, so re-apply the change.

The Drift version includes a more recent version of Netty, which adds TLS 1.3 support by default (among other things). This will allow Drift clients and servers to establish and accept TLS 1.3 connections where they wouldn't before.

== RELEASE NOTES ==

General Changes

Add support for TLS 1.3 in Drift

@sotodel sotodel requested a review from a team as a code owner June 22, 2023 21:41
@sotodel sotodel requested a review from presto-oss June 22, 2023 21:41
@ajaygeorge
Copy link
Contributor

@sotodel Can you please remove second attempt from the commit message.
You can keep the commit message from a433154

This commit updates Drift version to 1.36 and Netty version
to 4.1.92.Final to add support for TLS 1.3.
Copy link
Contributor

@ajaygeorge ajaygeorge left a comment

Choose a reason for hiding this comment

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

LGTM

@ajaygeorge ajaygeorge merged commit 5919a5a into prestodb:master Jun 26, 2023
@wanglinsong wanglinsong mentioned this pull request Jul 27, 2023
28 tasks
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.

2 participants