You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by GitBox <gi...@apache.org> on 2020/12/07 15:41:37 UTC

[GitHub] [avro] jturkel opened a new pull request #1038: AVRO-3000: Avoid unnecessary schema compatibility checks

jturkel opened a new pull request #1038:
URL: https://github.com/apache/avro/pull/1038


   Ruby Avro decoding spends a fair amount of time validating that the
   reader and writer schemas are compatible. These checks are
   unnecessary for the fairly common case of the reader and writer schemas
   being the same Avro::Schema instance. This improves the throughput
   of our Avro decoding benchmarks by 1.3X.
   
   While I was in here I removed repeated allocations of constant arrays in the `Avro:: SchemaCompatibility` module.
   
   /cc @tjwp 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [avro] tjwp merged pull request #1038: AVRO-3000: Avoid unnecessary schema compatibility checks

Posted by GitBox <gi...@apache.org>.
tjwp merged pull request #1038:
URL: https://github.com/apache/avro/pull/1038


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [avro] jturkel commented on pull request #1038: AVRO-3000: Avoid unnecessary schema compatibility checks

Posted by GitBox <gi...@apache.org>.
jturkel commented on pull request #1038:
URL: https://github.com/apache/avro/pull/1038#issuecomment-753695945


   @tjwp - I rebased this to pick up the switch to use Github Actions for CI. Looks like tests are passing now 🎉 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org