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


   > Just to clarify @eladkal - you think the proposed change is ok :) ? I am 
about to start preparing the providers release so we should merge that one I 
guess ? Sorry i have been swamped a bit recently and had no tone to look in 
detail
   
   The current PR doesnt solve the issue. It just hide warnning. The solution 
is doing a small refactor as suggested so get_client_type(IAM) is not needed. 
To my understanding this is what @dimon222 is going to do? (Then we dont need 
to bump major version)


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