mobuchowski commented on code in PR #39614:
URL: https://github.com/apache/airflow/pull/39614#discussion_r1606073804
##########
airflow/providers/openlineage/utils/utils.py:
##########
@@ -382,13 +382,6 @@ def get_filtered_unknown_operator_keys(operator:
BaseOperator) -> dict:
return {attr: value for attr, value in operator.__dict__.items() if attr
not in not_required_keys}
-def normalize_sql(sql: str | Iterable[str]):
Review Comment:
If you remove _anything_ from OpenLineage provider, you need to bump minimum
dependencies of anything relying on this version. Or, better, don't remove it
yet, just remove all usage and mark as deprecated.
--
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]
