dstandish opened a new pull request, #36227:
URL: https://github.com/apache/airflow/pull/36227

   I am trying to clean up KPO logging logic a bit.
   
   I was trying to better understand the logic in 
https://github.com/apache/airflow/pull/34412.  It occurred to me that one way 
to understand what it does, is to remove the logic and then run the new tests 
-- then I could compare old behavior vs new behavior.
   
   But so I did that (as you can see in this PR).  But then I found that the 
tests still passed!  So either the tests do not test the new behavior, or the 
changes don't do anything! 


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to