potiuk commented on PR #44533: URL: https://github.com/apache/airflow/pull/44533#issuecomment-2567576961
> I don't see why the two failing tests are failing - they are really unrelated to the PR - but they are green on canary builds. I think the comment from @kaxil requires some double check, else from point of data and migration I am good with it. Those tests are run with `full tests needed` so they are very likely going to fail in `canary` (and for others tests as well after this one is merged. @vatsrahul1001 - if you can rebase it again and ping me when it fails, I might want to have a deeper look or maybe @dabla might take a look. It definitely comes from https://github.com/apache/airflow/pull/41327 becasue that's where `get_primary_keys` was introduced. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
