eladkal commented on pull request #19725:
URL: https://github.com/apache/airflow/pull/19725#issuecomment-976175288


   > however, we see that `iam` case that potentially wasn't considered (most 
likely just accidentally missed).
   
   It was missed
   
   > iam scenario becomes "corner case" like it or not. If we want to polish 
that fragment we could consider stopping calls to respective method for iam, 
this way avoiding any corner case considerations for method's logic.
   
   This is probably the better solution as 
https://github.com/apache/airflow/pull/17987 solves a real problem so 
apparently it's just incomplete. 
   
   


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