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

[CASSANDRA-19814] UDT codec decode is too restrictive in decoding of unknown fields #1942

Open
wants to merge 2 commits into
base: 4.x
Choose a base branch
from

Conversation

netudima
Copy link

@netudima netudima commented Aug 3, 2024

Make UdtCodec logic tolerant to extra unknown fields at the tail of UDT to support live schema upgrades

Patch by Dmitry Konstantinov; reviewed by TBD for CASSANDRA-19814

…DT to support live schema upgrades

Patch by Dmitry Konstantinov; reviewed by TBD for CASSANDRA-19814
Patch by Dmitry Konstantinov; reviewed by TBD for CASSANDRA-19814
@tolbertam
Copy link
Contributor

Hi @netudima , apologies that this went under my radar. This looks similar to #1635, where there is some debate about whether this should be allowed (I am in support of the change as I think it's safe). I'll push to continue the conversation there. Would appreciate any commentary/feedback there!

@netudima
Copy link
Author

@tolbertam Thank you a lot for the link! I will check the conversation.

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