mobuchowski commented on code in PR #43936:
URL: https://github.com/apache/airflow/pull/43936#discussion_r1848352090
##########
providers/src/airflow/providers/openlineage/utils/utils.py:
##########
@@ -660,11 +660,11 @@ def decorator(f):
def wrapper(*args, **kwargs):
try:
return f(*args, **kwargs)
- except Exception as e:
+ except Exception:
log.warning(
- "Note: exception below is being caught: it's printed for
visibility. However OpenLineage events aren't being emitted. If you see that,
task has completed successfully despite not getting OL events."
+ "Note: exception below is being caught: it's printed for
visibility. However OpenLineage events aren't being emitted. If you see that,
task has completed successfully despite not getting OL events.",
Review Comment:
Fixed.
##########
providers/src/airflow/providers/openlineage/utils/utils.py:
##########
@@ -660,11 +660,11 @@ def decorator(f):
def wrapper(*args, **kwargs):
try:
return f(*args, **kwargs)
- except Exception as e:
+ except Exception:
log.warning(
- "Note: exception below is being caught: it's printed for
visibility. However OpenLineage events aren't being emitted. If you see that,
task has completed successfully despite not getting OL events."
+ "Note: exception below is being caught: it's printed for
visibility. However OpenLineage events aren't being emitted. If you see that,
task has completed successfully despite not getting OL events.",
Review Comment:
Fixed.
--
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]
