xinbinhuang commented on a change in pull request #14554:
URL: https://github.com/apache/airflow/pull/14554#discussion_r585867407
##########
File path: airflow/providers/apache/beam/provider.yaml
##########
@@ -41,6 +41,3 @@ hooks:
- integration-name: Apache Beam
python-modules:
- airflow.providers.apache.beam.hooks.beam
-
-hook-class-names:
- - airflow.providers.apache.beam.hooks.beam.BeamHook
Review comment:
I think it's a bit tricky for `BeamHook` to have those attributes
because Beam doesn't tie to a specified "backend", it can be run on Dataflow,
Flink, Spark, etc. and so their credentials may also come in different forms. I
think it's possible, but it may require some extra thoughts on what the
`Connection` object for Beam will look like. For now, we can at least get rid
of those warnings whenever `ProvidersManager` is instantiated.
----------------------------------------------------------------
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:
[email protected]
