kaxil commented on PR #73776:
URL: https://github.com/apache/airflow/pull/73776#issuecomment-5939760849

   Closing this. The description says it also changes `process_unparsed_files`, 
but no function by that name exists in Airflow and the diff touches only 
`process_parse_results`. There are no tests, and the description says the test 
suite wasn't run (only `py_compile`). When there is no earlier stat, the change 
writes `finish_time` into a callback-only stat, which the comment on main says 
must not happen. The diff also de-indents every parameter of 
`process_parse_results` to column 0.
   
   #73433 already addresses #73432, with tests.


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

Reply via email to