snazzyfox commented on issue #8933:
URL: https://github.com/apache/airflow/issues/8933#issuecomment-636354348
Glad to hear that system tests will be coming! We should definitely still
add it to the dependencies because that would actually fix it. System tests
would only help catch the
snazzyfox commented on issue #8933:
URL: https://github.com/apache/airflow/issues/8933#issuecomment-632345025
Yep, but `pyhive` is still needed for Hive. But since airflow is installing
it without hive support, it's not actually usable unless packages for hive
support are installed
snazzyfox commented on issue #8933:
URL: https://github.com/apache/airflow/issues/8933#issuecomment-631786295
Yep, pyhive supports both hive and presto, so depending on the user's needs
either `pyhive[hive]` or `pyhive[presto]` is installed.
Since airflow is just installing `pyhive`