mebelousov commented on issue #10641:
URL: https://github.com/apache/airflow/issues/10641#issuecomment-692009371
@mik-laj good idea! Thank you! Seems it's a good work-around.
I found out than method `_generate_insert_sql` in PostgresHook [1] is
incompatible with Greenplum syntax [2]. Now Greenplum doesn't allow to `ON
CONFLICT ({0}) DO UPDATE SET {1}`.
I believe to add GreenplumHook for full compatibility. GreenplumHook will be
copy of PostgresHook with exeption of `_generate_insert_sql`. It's enough to
use `_generate_insert_sql` from DbApiHook.
Is it good idea to add a new hook?
In general we have two choices:
1. Add GreenplumHook.
1. Use connection description.
---
1.
https://github.com/apache/airflow/blob/eaa49b2257913c34b15408a14e445f6106e691ee/airflow/providers/postgres/hooks/postgres.py#L194
2. http://docs.greenplum.org/6-0/ref_guide/sql_commands/INSERT.html
----------------------------------------------------------------
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:
[email protected]
