Maksim
Maksim
> Yeah 45K lines of .csv file is NOT something we want. Few options: > > 1. what happens when you zip the file ? how big it is going...
> Can we compress it (and dynamically decompress during test?). Just zipping it is 20K instead of 160K. This file is unlikely to ever change and it is cimpletely uninteresting...
> why don't you use the internal retry parameter of airflow ? I use the same approach what we use for retry Pod creation: https://github.com/apache/airflow/blob/main/airflow/providers/cncf/kubernetes/utils/pod_manager.py#L347C1-L356C1 Also, I didn't see any...
> Could you add tests to cover these retries? Sure, I have added a unit tests.
Hello Team, I see several issues which we need to fix for finishing this PR. 1. Fix unit tests which use `app`. After connexion update `create_app` function returns `connexion_app` instead...
@potiuk @ephraimbuddy do we need to do a db migration for this PR? In this PR db schema was changed. And previously for the 2.10.x version I have made it....
> > > generate migration file. Is it relevant to generate a migration file for Airflow 3.0.0? > > > > > > I think the idea is that all...
@potiuk @Lee-W I have updated the code.
> > It is needed to fix the issue in acknowledged functionality in deferrable mode. > > If this is a bug fix then we should have a unit test...
Hi @josh-fell @dstandish could you please look on this PR?