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 2021/12/05 22:13:38 UTC

[GitHub] [airflow] kazanzhy opened a new pull request #20058: Draft: Add Qubole how to documentation

kazanzhy opened a new pull request #20058:
URL: https://github.com/apache/airflow/pull/20058


   Add draft of Qubole how-to documentation for Qubole Operators and Sensors
   I will be very glad to any suggestions, especially from Qubole users.
   
   closes: #10929


-- 
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] kazanzhy commented on pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
kazanzhy commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986885485


   Did it, thanks
   Just trying to figure out what's wrong


-- 
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 pull request #20058: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-987210244


   Hey @kazanzhy - small comment :). Could you please use Github's "Draft" feature instaead of adding Draft: in the subject? 
   
   https://github.blog/2019-02-14-introducing-draft-pull-requests/
   
   The side effect of changing the subject is that if you are subscribed to GitHub notifications via email, the thread gets split.


-- 
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 pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986753099


   BTW @kazanzhy. I STRONGLY recommend `pip install pre-commit`. It saves a TON of time on iterations like that :).
   See https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst#id3


-- 
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 edited a comment on pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986753099


   BTW @kazanzhy. I STRONGLY recommend `pip install pre-commit` followed by `pre-commit install`. It saves a TON of time on iterations like that :).
   See https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst#id3


-- 
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] kazanzhy commented on pull request #20058: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
kazanzhy commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-987218042


   Copy that. Sorry for the bad habit from Gitlab :)


-- 
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] kazanzhy commented on pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
kazanzhy commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986725374


   Just fixed some issues with doc build.
   Now local doc building works well


-- 
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 pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986310341


   Nice @chattarajoy  waiting for your review to merge :)


-- 
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 pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986741155


   some provider.yaml inconsistency :) 


-- 
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 edited a comment on pull request #20058: Draft: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on pull request #20058:
URL: https://github.com/apache/airflow/pull/20058#issuecomment-986753099


   BTW @kazanzhy. I STRONGLY recommend `pip install pre-commit` followed by 'pre-commit install'. It saves a TON of time on iterations like that :).
   See https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst#id3


-- 
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 merged pull request #20058: Add Qubole how to documentation

Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #20058:
URL: https://github.com/apache/airflow/pull/20058


   


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