Skip to content

Conversation

@meszinorbi
Copy link
Contributor

No description provided.

@meszinorbi meszinorbi force-pushed the protobuf-shaded branch 2 times, most recently from 23459dd to bbbb6c6 Compare September 10, 2025 13:55
@meszinorbi meszinorbi changed the title OMID-319 Upgrade protobuf-java to 4.32.0 due to CVE-2024-7254 OMID-319 Use shaded protobuf-java from phoenix-thirdparty Sep 18, 2025

import java.util.List;

/**
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a note explaining where this was copied from and why

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added comment


import static io.netty.buffer.Unpooled.wrappedBuffer;

/**
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a note explaining where this was copied from and why

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added comment

@meszinorbi meszinorbi force-pushed the protobuf-shaded branch 2 times, most recently from 4193650 to 190d9f7 Compare October 20, 2025 14:32
@stoty
Copy link
Contributor

stoty commented Dec 10, 2025

Why aren't the github actions tests running ?

@stoty stoty marked this pull request as draft December 10, 2025 14:01
@stoty stoty marked this pull request as ready for review December 10, 2025 14:01
@stoty stoty merged commit 4899164 into apache:master Dec 11, 2025
1 check passed
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