pjfanning commented on PR #7128: URL: https://github.com/apache/hadoop/pull/7128#issuecomment-2434854078
@steveloughran I can see why Avro need this packages property but the implementation makes things difficult for Hadoop. One option would be for the Hadoop code to set the property itself. It could reset the property, taking the existing value and appending the necessary Hadoop packages. If the property is empty, we would need to add the default setting too if we were to add some of our own. The default applied is: https://github.com/apache/avro/blob/main/lang/java/avro/src/main/java/org/apache/avro/specific/SpecificDatumReader.java#L41 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
