You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/07/10 03:50:04 UTC

[GitHub] [airflow] skadyan opened a new pull request, #24945: Added databricks_conn_id as templated field

skadyan opened a new pull request, #24945:
URL: https://github.com/apache/airflow/pull/24945

   Issue: https://github.com/apache/airflow/issues/24938
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk commented on pull request #24945: Added databricks_conn_id as templated field

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #24945:
URL: https://github.com/apache/airflow/pull/24945#issuecomment-1180504054

   > Maybe all of the Databricks operators would benefit from this templating across the board? Just thinking others might have a similar use case to yours but use different operators. Not required for this PR of course (but feel free to add if you'd like), just tossing the idea out there.
   
   Or maybe (just a thought) we could automatically make conn_id fields "templated" :) . That would require likely some trickery.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] josh-fell commented on pull request #24945: Added databricks_conn_id as templated field

Posted by GitBox <gi...@apache.org>.
josh-fell commented on PR #24945:
URL: https://github.com/apache/airflow/pull/24945#issuecomment-1180443026

   Maybe all of the Databricks operators would benefit from this templating across the board? Just thinking others might have a similar use case to yours but use different operators. Not required for this PR of course (but feel free to add if you'd like), just tossing the idea out there.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk merged pull request #24945: Added databricks_conn_id as templated field

Posted by GitBox <gi...@apache.org>.
potiuk merged PR #24945:
URL: https://github.com/apache/airflow/pull/24945


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org