belugabehr
belugabehr
Unit test will fail on this PR until AVRO-401 is addressed.
AVRO-4061 is now fixed. Added unit test for coverage.
@KalleOlaviNiemitalo and @martin-g - Another pass on this one please?
@martin-g Thanks so much for your support on these PRs. Here's another one that needs attention that has a positive performance impact.
@KalleOlaviNiemitalo @martin-g Thank you for the correspondence. I'm taking another look at this. It seems less than ideal that there is synchronized code paths for fast-reading Avro data. I am...
I am taking another pass at this one. The performance improvements are just too significant to pass up (11%+ in my local benchmarks). Here are my benchmarks: ``` # JMH...
One thing to note is that I had to change the `Map` type. The `class` key uses reference based equals/hashcode and so had to convert to String to get a...
@martin-g Thanks for the pass. Any ideas who else might be available to review?
For consideration.