kbobrien commented on issue #10505: URL: https://github.com/apache/airflow/issues/10505#issuecomment-687239454
I was about to open a ticket to support the `target_session_attrs` psycopg2 parameter but this ticket could cover that use case too. Is there really a need to limit the parameters supplied to `psycopg2.connect()` on the Airflow side, psycopg2 itself doesn't limit the parameters it passes through to the underlying library? Could I suggest that we remove the List of parameters supported by the Hook altogether and just pass through what users supply in the extra_json as kv's? Since this check doesn't fail hard it provides limited benefit to users who probably expect their extra parameters were passed down since they got no warnings or errors. ---------------------------------------------------------------- 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]
