MaksYermak commented on PR #57804:
URL: https://github.com/apache/airflow/pull/57804#issuecomment-3547432282

   > I think we have to try to find another way of solving it. It does not seem 
right to make "composer" specific change in the common testing framework. I do 
not yet know how to solve it, I did not look into much details but - similarly 
as we have no MWAA references - even in amazon provider, I don't think we have 
any composer references - not only in google provider, but **especially** in 
common code.
   > 
   > It feels like any "composer-specific setup" and behaviour should be 
injected from outside via environment variables for example.
   > 
   > Again, I am not sure exaclty how to do it but we should likely brainstorm 
about it.
   
   @potiuk thank you for your review!
   
   I have updated the code and moved this Composer's related code to the 
google's provider. Could you please review this code and let me know what do 
you think about it?


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