search

yu-iskw / dbt-airflow-macros

Dbt package for Apache Airflow inspired macros
https://hub.getdbt.com/yu-iskw/dbt_airflow_macros/latest/
Apache License 2.0
15 stars 3 forks source link

Use dbt variables as an alternative way to get execution_date #27

Open lhvubtqn opened 1 year ago

lhvubtqn commented 1 year ago

Hi,

I currently use dbt-rpc to control my dbt project over http-rpc, so passing an environment variable for each run is not possible.

I think this PR will benefit others who are using dbt-rpc also.

Thanks

lhvubtqn commented 1 year ago

@yu-iskw Hi, hope to hear from you soon

yu-iskw commented 1 year ago

@lhvubtqn Thank you for the pull request. I don't oppose the idea, but let me think if it is the best approach for a bit.

Aside from that, as you may know, the dbt-rpc plugin will be fully deprecated by the end of 2022. So, due to the another reason, I am still wondering whether or not we should support the feature.

https://docs.getdbt.com/reference/commands/rpc

lhvubtqn commented 1 year ago

@lhvubtqn Thank you for the pull request. I don't oppose the idea, but let me think if it is the best approach for a bit.

Aside from that, as you may know, the dbt-rpc plugin will be fully deprecated by the end of 2022. So, due to the another reason, I am still wondering whether or not we should support the feature.

https://docs.getdbt.com/reference/commands/rpc

Thank you @yu-iskw