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/10/23 11:12:25 UTC

[GitHub] [airflow] aakashnand opened a new pull request, #27213: Allow setting client tags for trino connection

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

   Setting client tags information in trino connection can be helpful for more advance resource management. This PR allows users to set these client tags
   
   Thanks @eladkal for suggesting this idea. Related/Similar another PR is here : https://github.com/apache/airflow/pull/27095
   
   > Thanks @aakashnand BTW it would be cool to also support client_tags 
   https://github.com/trinodb/trino-python-client/blob/b4bd746bb57fe0e1c70e8544fd15b0c07ec98d2d/trino/dbapi.py#L111


-- 
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 #27213: Allow setting client tags for trino connection

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

   Rebased to check


-- 
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] uranusjr commented on pull request #27213: Allow setting client tags for trino connection

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

   I pushed a commit to fix static check failures.


-- 
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 #27213: Allow setting client tags for trino connection

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


-- 
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] aakashnand commented on pull request #27213: Allow setting client tags for trino connection

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

   @uranusjr we both pushed almost same time 😅 


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