You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by el...@apache.org on 2021/08/30 07:12:41 UTC

[airflow] branch main updated: fix wrong documents around upgrade-check.rst (#17903)

This is an automated email from the ASF dual-hosted git repository.

eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new e4d3da1  fix wrong documents around upgrade-check.rst (#17903)
e4d3da1 is described below

commit e4d3da16bad66aa40549cedac325d02e6b6559a9
Author: eladkal <45...@users.noreply.github.com>
AuthorDate: Mon Aug 30 10:12:16 2021 +0300

    fix wrong documents around upgrade-check.rst (#17903)
---
 docs/apache-airflow/upgrade-check.rst | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/docs/apache-airflow/upgrade-check.rst b/docs/apache-airflow/upgrade-check.rst
index 0c94f36..d6c580d 100644
--- a/docs/apache-airflow/upgrade-check.rst
+++ b/docs/apache-airflow/upgrade-check.rst
@@ -110,7 +110,8 @@ A sample output as a result of a successful run of the upgrade check is shown be
 
     Problems:
 
-    1.  Using ``airflow.operators.python_operator.PythonOperator`` will be replaced by ``airflow.operators.python.PythonOperator``. Affected file:
+    1.  Please install ``apache-airflow-backport-providers-presto``
+    2.  Using ``airflow.hooks.presto_hook.PrestoHook`` will be replaced by ``airflow.providers.presto.hooks.presto.PrestoHook``. Affected file:
 
 
 The following sections describe what is being done and how to apply the recommendations shown above.
@@ -138,12 +139,12 @@ problematic airflow.cfg and searching for the fernet_key entries would show the
     fernet_key =
 
 The second problem was identified in one of the DAGs. In this case, this import
-statement for the ``PythonOperator`` needs to be changed, since the location is different
+statement for the ``PrestoHook`` needs to be changed, since the location is different
 in Airflow 2.0. Examining the DAG file would probably show the following:
 
 .. code-block:: python
 
-    from airflow.operators.python_operator import PythonOperator
+    from airflow.hooks.presto_hook import PrestoHook
 
 We will discuss how to fix these and make them compatible with Airflow 2.0 in the next
 section.
@@ -158,15 +159,13 @@ enough information to make the change.
 For the first problem identified above with respect to the fernet_key, the solution is
 to enter a valid value in the Airflow Configuration file airflow.cfg for the fernet_key.
 
-For the second problem, looking at the source of the DAG file and changing the import
-statement for the Python Operator to be as follows will make this DAG work in Airflow 2.0.
+For the second problem, you will need to install Presto backport provider.
+Then you are able to use provider imports which contains the updated version of the hook.
 
 .. code-block:: python
 
-    from airflow.operators.python import PythonOperator
+    from airflow.providers.presto.hooks.presto import PrestoHook
 
-However, at the time of writing, this is incompatible in Airflow 1.10.15. So, this change
-can only be made while moving to Airflow 2.0.
 
 
 Turning off checks