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/04/01 08:58:18 UTC

[GitHub] [airflow] ashb commented on a change in pull request #15100: Add support for arbitrary json in conn uri format

ashb commented on a change in pull request #15100:
URL: https://github.com/apache/airflow/pull/15100#discussion_r605489162



##########
File path: docs/apache-airflow/howto/connection.rst
##########
@@ -289,12 +274,63 @@ Additionally, if you have created a connection, you can use ``airflow connection
 
 .. _manage-connections-connection-types:
 
+Encoding arbitrary json

Review comment:
       ```suggestion
   Encoding arbitrary JSON
   ```

##########
File path: docs/apache-airflow/howto/connection.rst
##########
@@ -289,12 +274,63 @@ Additionally, if you have created a connection, you can use ``airflow connection
 
 .. _manage-connections-connection-types:
 
+Encoding arbitrary json
+^^^^^^^^^^^^^^^^^^^^^^^
+
+Some json structures cannot be urlencoded without loss.  For such json, ``get_uri``

Review comment:
       ```suggestion
   Some JSON structures cannot be urlencoded without loss.  For such JSON, ``get_uri``
   ```




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