eolivelli commented on pull request #10878:
URL: https://github.com/apache/pulsar/pull/10878#issuecomment-858386478


   @codelipenghui we can follow up on #10888
   
   You can already use getSchemaType() in order to check if this is a 
KeyValueSchema
   
   Adding those methods to the Schema API may not be a good idea for the API 
(thinking to the future), because we would add some new methods to the top 
level API but those methods will be useful only to the KeyValueSchema users.
   
   Also it may encourage developers to think that we can nest KeyValueSchema, 
but the KeyValueSchema is not strictly a simple KeyValue Pair it is something 
that is strictly related to message encoding.
   


-- 
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:
us...@infra.apache.org


Reply via email to