Update base classes used in bigquery_check_operator to renamed versions#10272
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 Contribution Guide (https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst)
|
|
@dewaldabrie Can you look here? #10271 |
|
@mik-laj Thanks, I replied there. Not sure why the checks in the CI is failing since it seems related to things I didn't touch like |
|
We have a bug in the CI cache specification. I am going to fix it shortly |
|
SHould be fixed now by #10299. I also cherry-picked it to 1-10-stable and 1-10-test. Can you please rebase again on top of the latest v1-10-stable and it should be fixed permanently now. |
|
Hey @dewaldabrie - if you want this one to be part of 1.10.12 (willl be released very shortly) you should rebase this one |
e262e0e to
73e1934
Compare
|
Awesome work, congrats on your first merged pull request! |
Co-authored-by: Dewald Abrie <dewald@rentalcover.com>
Co-authored-by: Dewald Abrie <dewald@rentalcover.com>
…he#10272) Co-authored-by: Dewald Abrie <dewald@rentalcover.com>
airflow.contrib.operators.bigquery_check_operator.BigQueryValueCheckOperatoris derived fromairflow.operators.check_operator.ValueCheckOperatorbut needs to be derived fromairflow.operators.sql.SQLValueCheckOperatorinstead.Until this is done, airflow generates a lot of warnings from this line of code.
Fixes #10271