[GitHub] [airflow] potiuk edited a comment on pull request #15589: [Oracle] Add port to DSN

2021-05-08 Thread GitBox
potiuk edited a comment on pull request #15589: URL: https://github.com/apache/airflow/pull/15589#issuecomment-835188482 Yeah. Can you please rebase on top of the latest master? There were a couple of problems in master that have been just fixed. -- This is an automated message from the

[GitHub] [airflow] potiuk edited a comment on pull request #15589: [Oracle] Add port to DSN

2021-04-29 Thread GitBox
potiuk edited a comment on pull request #15589: URL: https://github.com/apache/airflow/pull/15589#issuecomment-829428066 > So if we want host:port/ to be translated back to host:port/, then it seems conn.schema is the way to go. Hmm. I think this is not how (at least originally) the

[GitHub] [airflow] potiuk edited a comment on pull request #15589: [Oracle] Add port to DSN

2021-04-29 Thread GitBox
potiuk edited a comment on pull request #15589: URL: https://github.com/apache/airflow/pull/15589#issuecomment-829428066 > So if we want host:port/ to be translated back to host:port/, then it seems conn.schema is the way to go. Hmm. I think this is not how (at least originally) the

[GitHub] [airflow] potiuk edited a comment on pull request #15589: [Oracle] Add port to DSN

2021-04-29 Thread GitBox
potiuk edited a comment on pull request #15589: URL: https://github.com/apache/airflow/pull/15589#issuecomment-829428066 > So if we want host:port/ to be translated back to host:port/, then it seems conn.schema is the way to go. Hmm. I think this is not how (at least originally) the