potiuk commented on issue #8432: Provide GCP credentials in Bash/Python 
operators
URL: https://github.com/apache/airflow/pull/8432#issuecomment-615728611
 
 
   I disagree. It's not a matter about inheritance vs. composition. I think we 
can find a good solution for that - but It's the matter of bringing GCP 
specific code to shared Bash Operator. Not everyone uses GCP and having code 
that is GCP-specific in general-purpose operator hurts my eyes.
   
   BashOperator should know nothing about GCP.  Maybe indeed this should be 
done via plugins or similar solution.
   
   I wonder what others thing about it @turbaszek @kaxil @ashb ? Should 
BashOperator contain GCP-specific code? WDYT?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to