Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-26 Thread via GitHub 


potiuk commented on PR #60363:
URL: https://github.com/apache/airflow/pull/60363#issuecomment-3800566602

   I am closing all your PRs @Arunodoy18 - ydespite earlier warnings, you are 
not looking at your PRs, submit multiple unrelated, AI generated changes 
without even looking at them. If this continues to happen, we will ask ASF 
infra to block your user for any ASF contributions.


-- 
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]

Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-26 Thread via GitHub 


potiuk closed pull request #60363: Re-enable dag_processing.last_duration metric
URL: https://github.com/apache/airflow/pull/60363


-- 
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]

Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-11 Thread via GitHub 


Arunodoy18 commented on PR #60363:
URL: https://github.com/apache/airflow/pull/60363#issuecomment-3734353711

   Yeaah sure , Apologising for the blunder , I will correct it within some
   time.
   
   On Sun, Jan 11, 2026, 3:36 PM Yeonguk Choo ***@***.***> wrote:
   
   > *choo121600* left a comment (apache/airflow#60363)
   > 
   >
   > It looks like this PR also includes commits addressing a different issue.
   > Could we split those changes into a separate PR to keep the scope clear?
   >
   > —
   > Reply to this email directly, view it on GitHub
   > ,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you authored the thread.Message ID:
   > ***@***.***>
   >
   


-- 
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]

Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-11 Thread via GitHub 


choo121600 commented on PR #60363:
URL: https://github.com/apache/airflow/pull/60363#issuecomment-3734346164

   It looks like this PR also includes commits addressing a different issue.
   Could we split those changes into a separate PR to keep the scope clear?


-- 
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]

Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-10 Thread via GitHub 


Arunodoy18 commented on code in PR #60363:
URL: https://github.com/apache/airflow/pull/60363#discussion_r2679271719


##
airflow-core/newsfragments/60325.bugfix.rst:
##


Review Comment:
   Thanks for the feedback, @choo121600.
   
   I've removed the unnecessary 60325.bugfix.rst newsfragment.
   
   I also found and removed a duplicate StatsD metric for 
[dag_processing.last_duration](vscode-file://vscode-app/c:/Users/aruno/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/code/electron-browser/workbench/workbench.html)
 which was likely causing the CI failure.
   
   The CI should be passing now. Let me know if there is anything else.



-- 
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]

Re: [PR] Re-enable dag_processing.last_duration metric [airflow]

2026-01-10 Thread via GitHub 


choo121600 commented on code in PR #60363:
URL: https://github.com/apache/airflow/pull/60363#discussion_r2678599749


##
airflow-core/newsfragments/60325.bugfix.rst:
##


Review Comment:
   It looks like the CI is failing.
   Also, this PR seems to include some unnecessary changes. Could you please 
take a look?



-- 
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]