TheNeuralBit commented on a change in pull request #16558:
URL: https://github.com/apache/beam/pull/16558#discussion_r826219744



##########
File path: model/pipeline/src/main/proto/schema.proto
##########
@@ -37,6 +37,7 @@ message Schema {
   // REQUIRED. An RFC 4122 UUID.
   string id = 2;
   repeated Option options = 3;
+  repeated Option sdk_options = 5;

Review comment:
       Could be good to clarify that sdk_options will not impact equality of 
schemas. It's simply a convenience for user types. 




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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to