tmgstevens commented on PR #379: URL: https://github.com/apache/flume/pull/379#issuecomment-1209943824
Yes. The root cause appears to be that Avro is refusing to cast an int to a long, which causes the first serialisation to fail. Then, because of poor exception handling, the writer isn't closed and it fails on all subsequent messages. I've fixed both of those problems and then enhanced the unit tests to check any errors in the future, although the unit tests only fire if you give them Twitter credentials. Get Outlook for Android<https://aka.ms/AAb9ysg> ________________________________ From: Ralph Goers ***@***.***> Sent: Tuesday, August 9, 2022 7:23:06 PM To: apache/flume ***@***.***> Cc: Tristan Stevens ***@***.***>; Author ***@***.***> Subject: Re: [apache/flume] FLUME-3434: TwitterSource fails to cast int to long (PR #379) Does this resolve the problem with the TwitterSource or does the version still need to be upgraded? — Reply to this email directly, view it on GitHub<https://github.com/apache/flume/pull/379#issuecomment-1209722288>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABIIEFAGBJ4JWFNNANMOMQLVYKOYVANCNFSM56A5ZJ3Q>. You are receiving this because you authored the thread.Message ID: ***@***.***> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
