You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "moiseenkov (via GitHub)" <gi...@apache.org> on 2024/07/31 10:03:00 UTC
[PR] Bugfix query count statistics when parsing DAF file [airflow]
moiseenkov opened a new pull request, #41149:
URL: https://github.com/apache/airflow/pull/41149
This PR fixes a bug in counting queries during a DAG file parsing (the feature was implemented in #40323).
A context manager `count_queries` was implemented previously, and it is responsible for counting queries within its scope. The reason why it was missing some queries and misreporting statistics is that its scope wasn't covering all steps of parsing the DAG file. This PR moves a `count_queries` invocation higher to the level of `DagFileProcessor.process_file()` method, which ensures that all the queries are counted.
Relates to: #40916
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bugfix query count statistics when parsing DAF file [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #41149:
URL: https://github.com/apache/airflow/pull/41149
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bugfix query count statistics when parsing DAF file [airflow]
Posted by "moiseenkov (via GitHub)" <gi...@apache.org>.
moiseenkov commented on PR #41149:
URL: https://github.com/apache/airflow/pull/41149#issuecomment-2262782583
@jedcunningham , @ephraimbuddy , @potiuk , hi,
Could you please take a look at this PR?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org