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/09/11 13:49:45 UTC

[GitHub] [airflow] eladkal commented on pull request #17974: WIP: Implement before stm and after stm in DBApiHooks

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


   Should we as part of this change deprecate the `_generate_insert_sql` of  `PostgesSQLHook`?
   It supports `ON CONFLICT` with a unique parameter `replace_index` in `insert_rows`?
   
   https://github.com/apache/airflow/blob/4ee8f82d482974b634a81cb684703c01ef6ce7c4/airflow/providers/postgres/hooks/postgres.py#L235-L251


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