merlimat commented on issue #1917:  adding avro schema
URL: https://github.com/apache/incubator-pulsar/pull/1917#issuecomment-394903837
 
 
   > I don't think there is a path to do a validation for now, unless I am not 
wrong. @mgodave ? 
   
   I think there should be some kind of validation already for JSON. Also we 
need to validate that the schema is there. eg: topic is set to Avro<T>, a 
producer cannot connect and send "bytes". 
   
   Avro has a way to extract schema info. That is what should be stored in the 
registry. Otherwise the whole "registry" part is completely useless.
   
   > I think we should do the validation in a separate PR, more I think we need 
to think validation along with how SQL use schema registry.
   
   Yes, if we don't store the "type" info, we cannot getting that back and 
discover how the data looks like
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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