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

Update Avro version 1.11.3 #21174

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Update Avro version 1.11.3 #21174

merged 1 commit into from
Oct 18, 2023

Conversation

Akanksha-kedia
Copy link
Contributor

Screenshot 2023-10-17 at 5 23 09 PM

@Akanksha-kedia Akanksha-kedia requested a review from a team as a code owner October 17, 2023 12:11
@Akanksha-kedia
Copy link
Contributor Author

@skairali please review.

@skairali
Copy link
Member

I do not see any potential issue. Checking the code changes as it also has some code changes

skairali
skairali approved these changes Oct 18, 2023
View reviewed changes
Copy link
Member

@skairali skairali left a comment

Choose a reason for hiding this comment

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

Vulnerabilities emerging from existing avro versions (1.8.1, 1.9.0) used in the presto source code. Current change helps. All UT are passing

I am approving this PR

@skairali skairali merged commit 3d3ee49 into prestodb:master Oct 18, 2023
@wanglinsong wanglinsong mentioned this pull request Dec 8, 2023
Closed
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tdcmeehan tdcmeehan tdcmeehan approved these changes

@skairali skairali skairali approved these changes

@presto-oss presto-oss Awaiting requested review from presto-oss presto-oss is a code owner automatically assigned from prestodb/committers

Assignees

@skairali skairali

Labels
Security
Projects
Security
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Akanksha-kedia @skairali @tdcmeehan