Robert Bradshaw

R: @Polber

I updated to make str/bytes required. We should try to get this in the cut.

R: @ganesh4991

Now that we have flex templates, we prefer not propagating the ValueProvider boilerplate elsewhere without good reason, and advise flex templates for any template use that uses non-ValueProvider-enabled features. Is...

> > Now that we have flex templates, we prefer not propagating the ValueProvider boilerplate elsewhere without good reason, and advise flex templates for any template use that uses non-ValueProvider-enabled...

By default, all properties in a json schema are optional; to declare them otherwise one uses the required field: https://json-schema.org/understanding-json-schema/reference/object#required which we respect in Beam: https://github.com/apache/beam/blob/release-2.65/sdks/python/apache_beam/yaml/json_utils.py#L67 . This function takes...