jdarais opened a new pull request, #163: URL: https://github.com/apache/avro-rs/pull/163
This is a follow-up from my last PR: `SchemaAwareWriteSerializer` doesn't need the schema_stack field since it can make use of the call stack to track which schema it's currently using. Making this switch makes the schema tracking a bit more consistent, whereas before, sometimes the schema_stack was used, and sometimes a new `SchemaAwareWriteSerializer` as created. This switch also improves performance slightly, particularly when the serializer encounters union or reference values. Changes in this PR: - Remove schema_stack field in favor of `serialize_*_with_schema` member functions - Benchmark tests updated to include some union fields to measure performance of `serialize_*_with_schema` logic - Rename support classes to match SchemaAwareWriteSerializer -- 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: dev-unsubscr...@avro.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
