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 2021/03/09 19:54:28 UTC

[GitHub] [airflow] eladkal commented on pull request #14639: feat: add Trino provider

eladkal commented on pull request #14639:
URL: https://github.com/apache/airflow/pull/14639#issuecomment-794374091


   > > Presto hasn't an Operator itself, but some transfers/operators with other providers **PrestoToMySqlTransfer** and **PrestotoGCSOperator** are you planning to create them or the specific provider must add later?
   > 
   > Personally I would deprecate those and opt for a generic `TrinoOperator`. Transferring data with Trino can be done much more efficiently with CTAS queries. WDYT?
   
   This require that the target DB will be configured as connector within Trino isn't it? That may not always be possible. Sometimes you want to transfer to external resource.
   TrinoOperator is needed either case I believe so it will be easier to run queries (CTAS or others).


----------------------------------------------------------------
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:
users@infra.apache.org