apache / beam

Apache Beam is a unified programming model for Batch and Streaming data processing.
https://beam.apache.org/
Apache License 2.0
7.83k stars 4.24k forks source link

[Task]: Set beam schema proto field in user_type in typehint/schemas.py #29756

Open riteshghorse opened 10 months ago

riteshghorse commented 10 months ago

What happened?

Had to revert user_type._beam_schema_proto = _Ephemeral(schema) because of a failure caught in #29684. Add this again to generate correct documentation such that it doesn't affect the DecimalLogicalType in python.

https://github.com/apache/beam/blob/fa47640888cba41ab675b8a4bdb22b1d3a4b1fa1/sdks/python/apache_beam/typehints/schemas.py#L564

Issue Priority

Priority: 2 (default / most bugs should be filed as P2)

Issue Components

tvalentyn commented 10 months ago

tentatively assigning to @robertwb who worked on the original change and has context -- feel free to add details and unassign if you are not planning to work on it. Thank you!