[GitHub] [airflow] inytar commented on a change in pull request #7621: [AIRFLOW-6982] add native python exasol support

[GitBox <gi...@apache.org>]

inytar commented on a change in pull request #7621: [AIRFLOW-6982] add native python exasol support
URL: https://github.com/apache/airflow/pull/7621#discussion_r388949599
 
 

 ##########
 File path: airflow/providers/exasol/hooks/exasol.py
 ##########
 @@ -0,0 +1,179 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+from contextlib import closing
+
+import pyexasol
+from past.builtins import basestring
+
+from airflow.hooks.dbapi_hook import DbApiHook
+
+
+class ExasolHook(DbApiHook):
+    """
+    Interact with Exasol.
+    You can specify the pyexasol ``compression``, ``encryption``, ``json_lib``
+    and ``client_name``  parameters in the extra field of your connection
+    as ``{"compression": True, "json_lib": "rapidjson", etc}``.
+    See `pyexasol reference
+    <https://github.com/badoo/pyexasol/blob/master/docs/REFERENCE.md#connect>`_
+    for more details.
+    """
+    conn_name_attr = 'exasol_conn_id'
+    default_conn_name = 'exasol_default'
+    supports_autocommit = True
+
+    def __init__(self, *args, **kwargs):
+        super(ExasolHook, self).__init__(*args, **kwargs)
+        self.schema = kwargs.pop("schema", None)
+
+    def get_conn(self):
+        conn_id = getattr(self, self.conn_name_attr)
+        conn = self.get_connection(conn_id)
+        conn_args = dict(
+            dsn='%s:%s' % (conn.host, conn.port),
+            user=conn.login,
+            password=conn.password,
+            schema=self.schema or conn.schema)
+        # check for parameters in conn.extra
+        for arg_name, arg_val in conn.extra_dejson.items():
+            if arg_name in ['compression', 'encryption', 'json_lib', 'client_name']:
+                conn_args[arg_name] = arg_val
+
+        conn = pyexasol.connect(**conn_args)
+        return conn
+
+    def get_pandas_df(self, sql, parameters=None):
+        """
+        Executes the sql and returns a pandas dataframe
+
+        :param sql: the sql statement to be executed (str) or a list of
+            sql statements to execute
+        :type sql: str or list
+        :param parameters: The parameters to render the SQL query with.
+        :type parameters: mapping or iterable
+        """
+        with closing(self.get_conn()) as conn:
+            conn.export_to_pandas(sql, query_params=parameters)
+
+    def get_records(self, sql, parameters=None):
+        """
+        Executes the sql and returns a set of records.
+
+        :param sql: the sql statement to be executed (str) or a list of
+            sql statements to execute
+        :type sql: str or list
+        :param parameters: The parameters to render the SQL query with.
+        :type parameters: mapping or iterable
+        """
+        with closing(self.get_conn()) as conn:
+            with closing(conn.execute(sql, parameters)) as cur:
+                return cur.fetchall()
+
+    def get_first(self, sql, parameters=None):
+        """
+        Executes the sql and returns the first resulting row.
+
+        :param sql: the sql statement to be executed (str) or a list of
+            sql statements to execute
+        :type sql: str or list
+        :param parameters: The parameters to render the SQL query with.
+        :type parameters: mapping or iterable
+        """
+        with closing(self.get_conn()) as conn:
+            with closing(conn.execute(sql, parameters)) as cur:
+                return cur.fetchone()
+
+    def run(self, sql, autocommit=False, parameters=None):
+        """
+        Runs a command or a list of commands. Pass a list of sql
+        statements to the sql parameter to get them to execute
+        sequentially
+
+        :param sql: the sql statement to be executed (str) or a list of
+            sql statements to execute
+        :type sql: str or list
+        :param autocommit: What to set the connection's autocommit setting to
+            before executing the query.
+        :type autocommit: bool
+        :param parameters: The parameters to render the SQL query with.
+        :type parameters: mapping or iterable
+        """
+        if isinstance(sql, basestring):
+            sql = [sql]
+
+        with closing(self.get_conn()) as conn:
+            if self.supports_autocommit:
+                self.set_autocommit(conn, autocommit)
+
+            for query in sql:
+                self.log.info(query)
+                with closing(conn.execute(query, parameters)) as cur:
+                    self.log.info(cur.row_count)
+            # If autocommit was set to False for db that supports autocommit,
+            # or if db does not supports autocommit, we do a manual commit.
+            if not self.get_autocommit(conn):
+                conn.commit()
+
+    def set_autocommit(self, conn, autocommit):
+        """
+        Sets the autocommit flag on the connection
+
+        :param conn: Connection to set autocommit setting to.
+        :type conn: connection object
+        :param autocommit: The autocommit setting to set.
+        :type autocommit: bool
+        """
+        if not self.supports_autocommit and autocommit:
+            self.log.warning(
+                ("%s connection doesn't support "
+                 "autocommit but autocommit activated."),
+                getattr(self, self.conn_name_attr))
+        conn.set_autocommit(autocommit)
+
+    def get_autocommit(self, conn):
+        """
+        Get autocommit setting for the provided connection.
+        Return True if autocommit is set.
+        Return False if autocommit is not set or set to False or conn
+        does not support autocommit.
+
+        :param conn: Connection to get autocommit setting from.
+        :type conn: connection object
+        :return: connection autocommit setting.
+        :rtype bool
 
 Review comment:
   Thanks! I looked over the file some many times...

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


With regards,
Apache Git Services