Skip to content

Commit bf7bae4

Browse files
authored
Merge pull request #34409 Add warning comment on get_default_sql_provider that folks don't change this.
2 parents 02d53af + a4f4e1c commit bf7bae4

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

sdks/python/apache_beam/yaml/yaml_provider.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -661,6 +661,9 @@ def create_transform(self, type, args, yaml_create_transform):
661661
return self._transform_factories[type](yaml_create_transform, **args)
662662

663663

664+
# Note: This function is used to override the default provider by some
665+
# users, so a change here will be breaking to those users. Change with
666+
# caution.
664667
def get_default_sql_provider():
665668
return beam_jar(
666669
urns={'Sql': 'beam:external:java:sql:v1'},

0 commit comments

Comments
 (0)