Re: [PR] openlineage: adjust log levels [airflow]

2023-10-11 Thread via GitHub
kaxil merged PR #34801: URL: https://github.com/apache/airflow/pull/34801 -- 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: commits-unsubscr...@airflow.ap

Re: [PR] openlineage: adjust log levels [airflow]

2023-10-10 Thread via GitHub
JDarDagran commented on PR #34801: URL: https://github.com/apache/airflow/pull/34801#issuecomment-1755247637 Thanks @kaxil ! I've commited changes as suggested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] openlineage: adjust log levels [airflow]

2023-10-10 Thread via GitHub
JDarDagran commented on code in PR #34801: URL: https://github.com/apache/airflow/pull/34801#discussion_r1352318622 ## airflow/providers/openlineage/extractors/base.py: ## @@ -82,7 +82,7 @@ def extract(self) -> OperatorLineage | None: self.operator.__class__.__modul

Re: [PR] openlineage: adjust log levels [airflow]

2023-10-10 Thread via GitHub
kaxil commented on code in PR #34801: URL: https://github.com/apache/airflow/pull/34801#discussion_r1352313431 ## airflow/providers/openlineage/extractors/base.py: ## @@ -82,7 +82,7 @@ def extract(self) -> OperatorLineage | None: self.operator.__class__.__module__ +

[PR] openlineage: adjust log levels [airflow]

2023-10-06 Thread via GitHub
JDarDagran opened a new pull request, #34801: URL: https://github.com/apache/airflow/pull/34801 OpenLineage provider is reported to be a bit too verbose. This PR attempts to lower logging levels of some messages and also cleans up the code. -- This is an automated