Blackrobe opened a new issue #9252:
URL: https://github.com/apache/airflow/issues/9252


   **Description**
   
   `qds-sdk` already supports JupyterNotebookCommand 
(https://github.com/qubole/qds-sdk-py/blob/master/qds_sdk/commands.py#L1351) 
since version 1.15.0 (change notes here 
https://github.com/qubole/qds-sdk-py/releases/tag/V1.15.0)
   
   and Airflow through `requirements.txt` already includes `qds-sdk` version 
1.16.0 
https://github.com/apache/airflow/blob/master/requirements/requirements-python3.7.txt#L309
   
   but this hook 
https://github.com/apache/airflow/blob/master/airflow/providers/qubole/hooks/qubole.py#L26-L29
 (also L39-L50) has not been updated yet. 
   
   I suggest we add following the pattern on 
https://github.com/qubole/qds-sdk-py/blob/master/bin/qds.py#L39
   
   I already planned a fix 
https://github.com/apache/airflow/commit/785992d1dec6f7f837f1ffbb604535240e22d2bc
   but I have never do PR to Airflow before and I overwhelmed by the 
requirements before opening a PR :roll_eyes: could someone kindly guide me
   
   **Use case / motivation**
   
   I want Airflow's Qubole operator to be able to interact with Qubole Jupyter 
Notebooks. It's already in the Qubole SDK.


----------------------------------------------------------------
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:
[email protected]


Reply via email to