kaxil commented on a change in pull request #19035:
URL: https://github.com/apache/airflow/pull/19035#discussion_r738296073



##########
File path: docs/apache-airflow/tutorial.rst
##########
@@ -415,10 +415,9 @@ Let's break this down into 2 steps: get data & merge data:
 
 .. code-block:: python
 
-  from airflow.decorators import dag, task
-  from airflow.hooks.postgres import PostgresHook
-  from datetime import datetime, timedelta
   import requests
+  from airflow.decorators import task
+  from airflow.hooks.postgres_hook import PostgresHook

Review comment:
       ```suggestion
     from airflow.hooks.postgres import PostgresHook
   ```

##########
File path: docs/apache-airflow/tutorial.rst
##########
@@ -447,6 +446,10 @@ Here we are passing a ``GET`` request to get the data from 
the URL and save it i
 
 .. code-block:: python
 
+  from airflow.decorators import task
+  from airflow.hooks.postgres_hook import PostgresHook

Review comment:
       ```suggestion
     from airflow.hooks.postgres import PostgresHook
   ```

##########
File path: docs/apache-airflow/tutorial.rst
##########
@@ -475,10 +478,11 @@ Lets look at our DAG:
 
 .. code-block:: python
 
-  from airflow.decorators import dag, task
-  from airflow.hooks.postgres_hook import PostgresHook
   from datetime import datetime, timedelta
+
   import requests
+  from airflow.decorators import dag, task
+  from airflow.hooks.postgres_hook import PostgresHook

Review comment:
       ```suggestion
     from airflow.hooks.postgres import PostgresHook
   ```




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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to