You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2022/08/03 02:43:21 UTC

[GitHub] [rocketmq-schema-registry] humkum commented on pull request #25: refine avro serde type to SpecificAvroSerde and GernericAvroSerde

humkum commented on PR #25:
URL: https://github.com/apache/rocketmq-schema-registry/pull/25#issuecomment-1203419743

   > What is the relation between SpecificAvroSerde, GernericAvroSerde and AvroSerde?Up-down dependency, or parallel ?
   > 
   > If SpecificAvroSerde and GernericAvroSerde are exposed to users and they are dependent on AvroSerde, I think AvroSerde should be moved to common module, to avoid user confusion. If they are parallel, it is better to merge the Serde package and the Serializer package. Or I will feel a bit confused about which to choose.
   > 
   > @humkum @ferrirW
   
   I can't agree more with you, their relationship is an Up-down dependency, and I agree to move AvroSerde to common package


-- 
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-unsubscribe@rocketmq.apache.org

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