fix: wrong hook on SQL operator GenericTransfer#49783
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
|
@Baptistouille Thx for the fix, but I think it's best we add an assertion on the invoked hooks (e.g. source/destination) as well to avoid mistakes in the future as well. You can add the assertion of the test_paginated_read method in test_generic_transfer. |
|
@dabla Thanks for pointing out tests. |
|
Fixed in #50598 |
^ Add meaningful description above
Here is the implementation for the reported bug closes: #49731