Recent change to DbApiHook affects multiple providers #37838
Labels
area:CI
Airflow's tests and continious integration
area:core
kind:bug
This is a clearly a bug
needs-triage
label for new issues that we didn't triage yet
Apache Airflow version
2.8.2
If "Other Airflow 2 version" selected, which one?
No response
What happened?
It looks like #37246 has broken the mypy-providers test for multiple providers since they subclass DBApiHook and their insert_rows method signatures no longer match the super()
Providers affected:
Relevant portion of output from
breeze static-checks --all-files
What you think should happen instead?
I think the
insert_rows
method in each of those classes need to addexecutemany
to their signatures to match the parent class's signature, but I don't know enough about their particular usecases to know how they should default.How to reproduce
I noticed it while running
breeze static-checks --all-files
, I'm sure there are other faster/easier ways to repro it.Operating System
na
Versions of Apache Airflow Providers
No response
Deployment
Other
Deployment details
No response
Anything else?
No response
Are you willing to submit PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: