mik-laj edited a comment on pull request #20998: URL: https://github.com/apache/airflow/pull/20998#issuecomment-1028336645
@sungchun12 DBT is used quite often with Apache Airflow, so I found it worth asking for reviews to share your thoughts on this contribution. The earlier the changes are introduced, the lower the cost of their implementation. We also sometimes do not know all API features as they may not be widely promoted, but from your perspective they are important. For example, all requests to Google API contain client info, which allows them to track API usage by a specific solution https://github.com/apache/airflow/blob/d353f023ff8856c00b9f054526cb2e40ff0116ae/airflow/providers/google/common/hooks/base_google.py#L340-L352 Snowflake provider provides have a similar feature: https://github.com/apache/airflow/blob/d353f023ff8856c00b9f054526cb2e40ff0116ae/airflow/providers/snowflake/hooks/snowflake.py#L184 On the other hand, it can also be a signal for you that there is a new feature in the third project and you can promote it, e.g. by updating the documentation https://docs.getdbt.com/docs/running-a-dbt-project/running-dbt-in-production#using-airflow Does it make sense to you? -- 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]
