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

 ##########
 File path: pulsar-common/src/main/proto/PulsarApi.proto
 ##########
 @@ -22,6 +22,20 @@ package pulsar.proto;
 option java_package = "org.apache.pulsar.common.api.proto";
 option optimize_for = LITE_RUNTIME;
 
+message Schema {
+       required string name = 1;
+       required bytes version = 2;
+       required bytes schema_data = 3;
 
 Review comment:
   I tried to address this in a question posed on the original reference PR. I 
had originally specced this out with that intent but after much thought I 
decided not to add it. My intent was that the schema registry should just store 
a "Schema" no matter what it is, it does not need to know what it contains. The 
schema only really means something to the end systems (producer/consumer) and 
by extension the broker. We can allow the broker to "plug-in" modules to allow 
semantic parsing of the schema as needed. This way we can support any number of 
crazy business cases.

----------------------------------------------------------------
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