potiuk commented on a change in pull request #12466:
URL: https://github.com/apache/airflow/pull/12466#discussion_r532053560
##########
File path: airflow/provider.yaml.schema.json
##########
@@ -167,6 +170,13 @@
"python-module"
]
}
+ },
+ "hook-class-names": {
Review comment:
I wanted to explicity call out the Hook classes not modules. I think
this will be faster and better than trying to find Hooks and there might some
exceptions that we would have to handle (for example Hooks derived from other
Hooks). however I wanted to make some check and see how many those exceptions
we have and if we find that there are not many, we could indeed try to find all
the classes deriving from Base Hook in the "hook" modules specified. I will
take a look.
----------------------------------------------------------------
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]
