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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2019/03/05 05:16:37 UTC

[GitHub] [pulsar] merlimat edited a comment on issue #3752: revise the schema default type not null

merlimat edited a comment on issue #3752: revise the schema default type not null
URL: https://github.com/apache/pulsar/pull/3752#issuecomment-469541275
 
 
   Adding the flag and leave `allowNull=true` by default is good and easy. 
   
   I think though that using `allowNull=false` would be a better default option, but it doesn't have to be done right now in this PR. 
   
   Regarding the migration path, I'm not referring to migrate the schema definition, but rather to allow a one-time "incompatible" schema evolution marked with a version in pulsar schema registry.

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


With regards,
Apache Git Services