dstandish commented on PR #64571:
URL: https://github.com/apache/airflow/pull/64571#issuecomment-4605492023

   > Agree it might not worth the machinery for v1. The only reason I added it 
was for better discoverability. I can drop both to plain self.log and add it 
back if someone asks it. WDYT?
   
   I don't really have a strong feeling about that.  I tried to look for it in 
the code but could not find it.
   
   And I don't stand in the way of merging this thing....
   
   But... I can't help feeling that this PR is bigger than it should be.  It's 
feels so big that it's kind of overwhelming to properly review.  It spreads the 
reviewer's attention over a very wide area.
   


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