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/05/05 10:03:38 UTC

[GitHub] [airflow] pateash commented on pull request #15574: #12401 - Duplicating connections from UI

pateash commented on pull request #15574:
URL: https://github.com/apache/airflow/pull/15574#issuecomment-832567983


   > As others have said, we should have it as `_copy` not `_Copy`.
   > 
   > Additionally, it should be `copy1` not `copy(1)` so it can be set via env vars if _has_ to. (You can't put `()` in env var names, so `AIRFLOW_CONN_FOO_COPY(1)` would not be possible
   
   Thanks All for valuable comments,
   As per suggestions, I have used **_copyN**  Syntax for duplication,
   also test has been added.


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