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

AVRO-3786: [Rust] Deserialization results in FindUnionVariant error if the writer and reader have the same symbol but at different positions #2335

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

martin-g
Copy link
Member

AVRO-3786

What is the purpose of the change

  • Add the test case from the JIRA ticket.
  • Update the assertions to actually match the expected behavior.

Verifying this change

  • Execute the test suite

Documentation

  • Does this pull request introduce a new feature? yes

…f the writer and reader have the same symbol but at different positions

Add the test case from the JIRA ticket.
Update the assertions to actually match the expected behavior.

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
@github-actions github-actions bot added the Rust label Jul 10, 2023
@martin-g martin-g merged commit 8c3ee16 into master Jul 12, 2023
14 checks passed
martin-g added a commit that referenced this pull request Jul 12, 2023
…f the writer and reader have the same symbol but at different positions (#2335)

Add the test case from the JIRA ticket.
Update the assertions to actually match the expected behavior.

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
(cherry picked from commit 8c3ee16)
@martin-g martin-g deleted the avro-3786-enum-validation-2 branch July 12, 2023 06:13
RanbirK pushed a commit to RanbirK/avro that referenced this pull request May 13, 2024
…f the writer and reader have the same symbol but at different positions (apache#2335)

Add the test case from the JIRA ticket.
Update the assertions to actually match the expected behavior.

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant