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

Explain Avro's unnecessary EOFException (#4098) #4100

merged 2 commits into from Mar 24, 2017


None yet
3 participants

asdf2014 commented Mar 23, 2017

Explain Avro's unnecessary EOFException (#4098)

We should change the version of Avro from 1.7.7 into the 1.9.0 when it released, but right now we should explain it in log and fix it with patch, because it hurts the performance of Avro-Extension very.


This comment has been minimized.


gianm commented Mar 24, 2017

@asdf2014 thanks for the contribution! That's a strange avro bug. Could you include the JIRA link in the exception message too? That way, a user would see it without having to read the Druid code. 👍 from me after that.

Also, could you please fill out the CLA here:

@@ -87,6 +86,10 @@ public GenericRecord parse(ByteBuffer bytes)
try {
return, DecoderFactory.get().binaryDecoder(inputStream, null));
catch (EOFException eof) {
// waiting for avro v1.9.0 (#AVRO-813)
throw new ParseException("Avro's unnecessary EOFException", eof);

This comment has been minimized.


gianm Mar 24, 2017


Please include the JIRA link in this message and others like it.


This comment has been minimized.


asdf2014 commented Mar 24, 2017

@gianm Already add the jira link into log message. It's my pleasure, i have filled out the individual CLA.


This comment has been minimized.


himanshug commented Mar 24, 2017


@himanshug himanshug merged commit 23f77eb into apache:master Mar 24, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed

@gianm gianm added this to the 0.10.1 milestone Jul 6, 2017

@asdf2014 asdf2014 deleted the asdf2014:avro_eof branch Aug 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment