liferoad commented on code in PR #34708: URL: https://github.com/apache/beam/pull/34708#discussion_r2066296738
########## sdks/python/apache_beam/yaml/yaml_transform_unit_test.py: ########## @@ -38,8 +38,14 @@ from apache_beam.yaml.yaml_transform import preprocess_flattened_inputs from apache_beam.yaml.yaml_transform import preprocess_windowing from apache_beam.yaml.yaml_transform import push_windowing_to_roots +from apache_beam.yaml.yaml_transform import validate_against_schema from apache_beam.yaml.yaml_utils import SafeLineLoader +try: + import jsonschema +except ImportError: + jsonschema = None Review Comment: Do we need this since https://github.com/apache/beam/blob/b9fa49a9827dd28349e382f479ebd1a8bbe27d07/sdks/python/setup.py#L356 means jsonschema should be installed in general. -- 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-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
