Joffreybvn commented on code in PR #32304: URL: https://github.com/apache/airflow/pull/32304#discussion_r1249542691
########## airflow/providers/apache/impala/provider.yaml: ########## @@ -28,7 +28,7 @@ versions: - 1.0.0 dependencies: - - impyla>=0.18.0,<1.0 + - impyla[kerberos]>=0.18.0,<1.0 Review Comment: Yes, indeed, this will bother users that do not need it. My reasoning was the following: For Hadoop, [the provider is shipped with kerberos](https://github.com/apache/airflow/blob/main/airflow/providers/apache/hdfs/provider.yaml#L46). Thus, to stay consistent, for Impala (which is setup on top of an hadoop system), it makes sense to have it bundled too. I can propose a PR to add kerberos as optional dependency to hdfs and impala ? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
