TheNeuralBit commented on issue #22854: URL: https://github.com/apache/beam/issues/22854#issuecomment-1227771515
Hm actually the breakage is due to https://github.com/apache/beam/pull/22066, which is tougher to rollback. The core issue here is that PR started to mix beam typehints (RowTypeConstraint) and native typehints (typing.Optional). In all our test cases this was OK because the RowTypeConstraint was either the top-level, or if nested it was not nullable/optional, so it didn't need to be wrapped in a typing.Optional. -- 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]
