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 2020/01/12 07:28:15 UTC

[GitHub] [airflow] mik-laj edited a comment on issue #7142: [AIRFLOW-6539][AIP-21] Move Apache classes to providers.apache package

mik-laj edited a comment on issue #7142: [AIRFLOW-6539][AIP-21] Move Apache classes to providers.apache package
URL: https://github.com/apache/airflow/pull/7142#issuecomment-573389953
 
 
   Can you check if full compatibility has been maintained?
   
   For example:
   ```
   # pylint: disable=unused-import
   from airflow.providers.apache.pinot.hooks.pinot import PinotAdminHook, PinotDbApiHook  # noqa
   ```
   If `airflow.providers.apache.pinot.hooks.pinot` package contains  two classess, then two classes should be imported and two classes should be added in `test_core_to_contrib.py` file. The selection of classes was manual, because I did not want to maintain compatibility with BaseHook or other not important stuff.

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