merlimat commented on a change in pull request #1232: Schema registry (1/4)
URL: https://github.com/apache/incubator-pulsar/pull/1232#discussion_r168234826
 
 

 ##########
 File path: pulsar-common/src/main/proto/PulsarApi.proto
 ##########
 @@ -368,6 +387,7 @@ message CommandRedeliverUnacknowledgedMessages {
 
 message CommandSuccess {
        required uint64 request_id = 1;
+       optional Schema schema = 2;
 
 Review comment:
   The question there is that there can be multiple schemas valid for the topic 
a given point in time. Eg: when you rollout an upgrade in the producers they 
will start publishing a new format, but the old producers are still publishing. 
   
   I think when we talked about this, the solution was that the message itself 
will carry the version of the schema used so that the consumer can distinguish 
it. One option could be that you get a list of all valid schemas for the topic 
when subscribing, but there should also be a way for the consumer to retrieve 
new schema versions as they are updated.

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


With regards,
Apache Git Services

Reply via email to