chamikaramj commented on issue #22748: URL: https://github.com/apache/beam/issues/22748#issuecomment-1217287518
ID is probably set to empty string which is done here: https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/SchemaTranslation.java#L83 According to the proto, the ID has to be set to a UUID: https://github.com/apache/beam/blob/master/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/schema.proto#L40 So the actual fix might have to be done to the SchemaTranslation class (and any other locations where we do not set an ID when generating the Schema proto). I'll merge https://github.com/apache/beam/pull/22750 though to fix the tests. -- 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]
