xkrogen commented on PR #4580: URL: https://github.com/apache/hadoop/pull/4580#issuecomment-1187715602
I'm -1 on this. Avro versions follow the following convention (refer to [this comment](https://github.com/apache/avro/pull/1333#issuecomment-930355417) and [this dev list thread](https://lists.apache.org/thread/6ppm20v5602w9nqz0nk5qz7mxnnt2tsw)): ``` <data compatibility> . <major library version> . <minor library version> ``` So while going from 1.7.7 to 1.9.2 may look like a minor version upgrade at first glance, it's actually an upgrade of 2 major versions (7 -> 9). This is way too big of a change to include into a patch release of Hadoop. This isn't just a theoretical problem; Avro major version upgrades really do bring in breaking changes. We (LinkedIn) maintain [an entire library devoted to smoothing over such breaking changes](https://github.com/linkedin/avro-util/tree/master/helper). Speaking from personal experience, without such a compatibility layer, a major version upgrade of Avro will cause many downstreams to break. -- 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]
