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

Netty Version Upgrade to 4.1.115.Final for Drift #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

imsayari404
Copy link

@imsayari404 imsayari404 commented Dec 6, 2024

  1. Description:

This PR upgrades the Netty version to 4.1.115.Final and publishes the custom JAR to JitPack for the Drift project. The following changes were made:

  1. Motivation and context:

This upgrade ensures that we are using a more stable and secure version of Netty, addressing potential vulnerabilities in the previous version.

  1. Testing:

I ran mvn clean install, and the build was successful. CI tests will be triggered to validate these changes in the Presto draft PR with the updated Drift version.

image

- Upgraded `dep.netty.version` to 4.1.115.Final to include the latest fixes and improvements.
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 6, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 6, 2024
Copy link

linux-foundation-easycla bot commented Dec 7, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: imsayari404 / name: Sayari Mukherjee (deabf7d)

imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 7, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 7, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 7, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 9, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 11, 2024
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 20, 2024
-prestodb/drift pr : prestodb/drift#54
-Resolved presto-druid module error
-Resolving issues related to maven checks
-Ignored Specific duplicate resouces explicitly related to com/github/rvesse/airline/core.version
-Fix: Replace deprecated io.netty:netty with io.netty:netty-all version 4.1.115.Final
imsayari404 added a commit to imsayari404/presto that referenced this pull request Dec 20, 2024
-prestodb/drift pr : prestodb/drift#54
-Resolved presto-druid module error
-Resolving issues related to maven checks
-Ignored Specific duplicate resouces explicitly related to com/github/rvesse/airline/core.version
-Fix: Replace deprecated io.netty:netty with io.netty:netty-all version 4.1.115.Final
@imsayari404 imsayari404 marked this pull request as ready for review December 20, 2024 09:33
@imsayari404 imsayari404 requested a review from a team as a code owner December 20, 2024 09:33
Copy link
Member

@imjalpreet imjalpreet left a comment

Choose a reason for hiding this comment

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

Do we already have a draft presto PR that triggered the CI pipeline using JitPack dependency? If yes, can you also add the link to the draft PR here for reference?

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.

4 participants