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

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/03/14 17:41:19 UTC

[GitHub] [beam] TheNeuralBit commented on a change in pull request #16558: Add schemacoder and update row coder urn version

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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org