You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@avro.apache.org by GitBox <gi...@apache.org> on 2022/08/11 06:56:28 UTC

[GitHub] [avro] KalleOlaviNiemitalo commented on pull request #1819: AVRO-3603 dotnet reflect reader and writer refactoring

KalleOlaviNiemitalo commented on PR #1819:
URL: https://github.com/apache/avro/pull/1819#issuecomment-1211616306

   Does this change help users of the library in some way? Better support for unloadable assemblies perhaps?
   
   The breaking changes are worrisome, but I'm not using Avro.Reflect directly and <https://github.com/confluentinc/confluent-kafka-dotnet/tree/v1.9.2/src/Confluent.SchemaRegistry.Serdes.Avro> doesn't appear to use it either (uses only Avro.Generic and Avro.Specific).
   
   Can the dependency on Microsoft.Extensions.DependencyInjection be replaced with Microsoft.Extensions.DependencyInjection.Abstractions? For people using Autofac or other DI libraries.
   
   Can Avro.Reflect be split to a separate NuGet package so that users of Avro.Generic and Avro.Specific don't need to ship any DI libraries (and audit compliance with their licenses)?


-- 
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: issues-unsubscribe@avro.apache.org

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