potiuk commented on code in PR #28538: URL: https://github.com/apache/airflow/pull/28538#discussion_r1055825466
########## airflow/providers/apache/hive/plugins/hive_plugin.py: ########## @@ -0,0 +1,42 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment: Should we also remove them from "airbyte", "arango", "...." (just checked the first few)? This is for providers that have multiple modules in their packages. Most of the providers of ours have just a single module - and all classes are in this module: ``` airbyte.hooks.airbyte airbyte.operators.airbyte airbyte.sensors.airbyte arangodb.hooks.arangodb arangodb.operators.arangodb arangodb.sensors.arangodb .... tableau.hooks.tableau tableau.operators.tableau tableau.sensors.tableau ``` Should we change those too? Are we calling for a new AIP? -- 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]
