[GitHub] [airflow] dstandish edited a comment on pull request #15100: Add support for arbitrary json in conn uri format

2021-03-31 Thread GitBox
dstandish edited a comment on pull request #15100: URL: https://github.com/apache/airflow/pull/15100#issuecomment-811659712 ok so to be clear @mik-laj you are in support of supporting arbitrary dict under `extra`, just not primative or list? OR, you do _not_ support this PR at all an

[GitHub] [airflow] dstandish edited a comment on pull request #15100: Add support for arbitrary json in conn uri format

2021-03-31 Thread GitBox
dstandish edited a comment on pull request #15100: URL: https://github.com/apache/airflow/pull/15100#issuecomment-811659712 ok so to be clear @mik-laj you are in support of supporting arbitrary dict under `extra`, just not primative or list? OR, you do _not_ support this PR at all an

[GitHub] [airflow] dstandish edited a comment on pull request #15100: Add support for arbitrary json in conn uri format

2021-03-31 Thread GitBox
dstandish edited a comment on pull request #15100: URL: https://github.com/apache/airflow/pull/15100#issuecomment-811659712 ok so to be clear @mik-laj you are in support of supporting arbitrary dict under `extra`, just not primative or list? OR, you do _not_ support this PR at all an

[GitHub] [airflow] dstandish edited a comment on pull request #15100: Add support for arbitrary json in conn uri format

2021-03-30 Thread GitBox
dstandish edited a comment on pull request #15100: URL: https://github.com/apache/airflow/pull/15100#issuecomment-810654779 > I like this solution with the use of a special key. I don't understand the role of base64 here. Is it necessary? It seems to me that the pure value will be easier t

[GitHub] [airflow] dstandish edited a comment on pull request #15100: Add support for arbitrary json in conn uri format

2021-03-30 Thread GitBox
dstandish edited a comment on pull request #15100: URL: https://github.com/apache/airflow/pull/15100#issuecomment-810654779 > I like this solution with the use of a special key. I don't understand the role of base64 here. Is it necessary? It seems to me that the pure value will be easier t