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 2022/03/13 14:55:45 UTC

[GitHub] [airflow] potiuk opened a new issue #22220: Databricks SQL fails on Python 3.10

potiuk opened a new issue #22220:
URL: https://github.com/apache/airflow/issues/22220


   ### Apache Airflow Provider(s)
   
   databricks
   
   ### Versions of Apache Airflow Providers
   
   The databricks SQL does not work on Python 3.10 due to "from collections import Iterable" in the `databricks-sql-connector`
   
   * https://pypi.org/project/databricks-sql-connector/
   
   Details of this issue dicussed in https://github.com/apache/airflow/pull/22050
   
   For now we will likely just exclude the tests (and mark databricks provider as non-python 3.10 compatible). But once this is fixed (in either 1.0.2 or upcoming 2.0.0 version of the library, we wil restore it back). 
   
   ### Apache Airflow version
   
   main (development)
   
   ### Operating System
   
   All
   
   ### Deployment
   
   Other
   
   ### Deployment details
   
   Just Breeze with Python 3.10
   
   ### What happened
   
   The tests are failing:
   
   ```
     self = <databricks.sql.common.ParamEscaper object at 0x7fe81c6dd6c0>
     item = ['file1', 'file2', 'file3']
     
         def escape_item(self, item):
             if item is None:
                 return 'NULL'
             elif isinstance(item, (int, float)):
                 return self.escape_number(item)
             elif isinstance(item, basestring):
                 return self.escape_string(item)
     >       elif isinstance(item, collections.Iterable):
     E       AttributeError: module 'collections' has no attribute 'Iterable'
     ```
   
   https://github.com/apache/airflow/runs/5523057543?check_suite_focus=true#step:8:16781 
   
   
   ### What you expected to happen
   
   Test succeed :)
   
   ### How to reproduce
   
   Run `TestDatabricksSqlCopyIntoOperator` in Python 3.10 environment.
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] potiuk commented on issue #22220: Databricks SQL fails on Python 3.10

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #22220:
URL: https://github.com/apache/airflow/issues/22220#issuecomment-1066119096


   cc: @alexott  - I created an issue to track it (and proceed with excluding the failing test in Python 3.10 for now so that it does not block us from enabling 3.10 for Airflow main CI.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org