sijie commented on issue #6405: Add verification for 
SchemaDefinitionBuilderImpl.java
URL: https://github.com/apache/pulsar/pull/6405#issuecomment-590517434
 
 
   @codelipenghui I don't think this is a good fix. I think we should verify 
the schema definition if both schema def and POJO are provided. There are two 
ways to get the schema definition, one is to retrieve the schema information 
from the generated AVRO code or get the json schema definition from POJO by 
reflection. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to