[#40] Support configurable endianness on schema id, update and clean doc

agile-lab-dev / darwin

Avro Schema Evolution made easy

Apache License 2.0

34 stars 10 forks source link

[#40] Support configurable endianness on schema id, update and clean doc #42

Closed tmnd1991 closed 5 years ago

tmnd1991 commented 5 years ago

Mind giving a look guys? I'm quite confident, but 2 * n eyes with n > 1 is much better than only 2

tmnd1991 commented 5 years ago

I thought about making little_endian the default, but for now, I think that is good to make the user explicitly define one. This is because big_endian was the default and only available as of 1.0.8 and simply upgrading without adding the configuration would inevitably break a lot of projects.

erond commented 5 years ago

+1 for the name of the property and for the thoughts about not making "little_endian" the default so not to break backward compatibility up in legacy projects.