drobert commented on a change in pull request #19053:
URL: https://github.com/apache/airflow/pull/19053#discussion_r732250913
##########
File path: docs/apache-airflow/tutorial.rst
##########
@@ -446,13 +469,14 @@ Here we are passing a ``GET`` request to get the data
from the URL and save it i
from "Employees_temp";
"""
try:
- postgres_hook = PostgresHook(postgres_conn_id="LOCAL")
+ postgres_hook = PostgresHook(postgres_conn_id=conn_id)
conn = postgres_hook.get_conn()
cur = conn.cursor()
cur.execute(query)
conn.commit()
return 0
except Exception as e:
+ logging.error(f"Failed to merge data: {e}")
Review comment:
I had point 1
https://github.com/apache/airflow/pull/19053/files#diff-5d885a5057f9caeddb7c96e09d905011e8bb58024009d1f9badaa37dee9ab283R504
but I think I keep losing context during copy-paste between the 'full code'
and smaller examples. I'll take another pass.
Thanks, re point 2. I'm new to python and thought I was doing that
correctly, but will remove.
--
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]
