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 null deserialization #216

Closed
marcospassos opened this issue Jul 28, 2020 · 9 comments
Closed

Avro null deserialization #216

marcospassos opened this issue Jul 28, 2020 · 9 comments
Milestone

Comments

@marcospassos
Copy link
Contributor

We found another issue related to schema evaluation, after deserialization of a null value. After a few hours of debugging, we found that the root issue has already been fixed, but it was not included in the latest release.

Is there any particular reason not to release this fix for 2.11.x versions?

@cowtowncoder
Copy link
Member

Looking at #146 PR (for #145) originally submitted against master; I would not be against fix for 2.x if anyone has time to back port the change.

Since there is a new feature, it would have to go in 2.12(.0) however.

@marcospassos
Copy link
Contributor Author

Is it ok if I provide a PR including only the fix for the null serialization issue?

@marcospassos
Copy link
Contributor Author

PR opened.

@cowtowncoder
Copy link
Member

Yes that makes sense.

@cowtowncoder cowtowncoder added this to the 2.11.2 milestone Jul 28, 2020
cowtowncoder added a commit that referenced this issue Jul 28, 2020
@marcospassos
Copy link
Contributor Author

Any ETA for 2.11.2?

@cowtowncoder
Copy link
Member

I was hoping to hold off for bit longer as 2.11.1 was only released 1 month ago and total number of fixes is still quite low. But there are couple of important fixes so I think I will release this within a week, maybe over the weekend.

@marcospassos
Copy link
Contributor Author

Thank you @cowtowncoder :)

@cowtowncoder
Copy link
Member

2.11.2 was released on August 2nd, 2020.

@marcospassos
Copy link
Contributor Author

Thank you @cowtowncoder.

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

No branches or pull requests

2 participants