Vincent
Vincent
Moving the example dags in tests directory will not make the CI run them so we are safe on that side. Though I am wondering if we want to have...
Not a big deal but you could have split this PR in 2; Bedrock and Opensearch. Especially because you do not mention opensearch in the PR title.
> I'm really not sure what that failing static check is trying to tell me. Wouldn't mind a hand on that if someone knows what it means. I think, it...
Thanks @mxuan0 for posting the issue, it is indeed a bug, here is a fix: #39671
LGTM
> I was thinking about the BasOperator argument `retries` > > ```python > PythonOperator( > task_id="aa", > retries=3, > python_callable=toto, > ) > ``` This is an option for users....
I just added few comments. The code changes look massive, some changes do not look related to the PR. Could you please rebase your PR against main? Also, I think...
Thanks for picking up this effort though!
> > This was my first time to use rebase. > > The first time with `git rebase` it ever painful, after that things become much easier 💯
> Hi @vincbeck we rebased the branch with `main`. Still, the PR is massive because this PR is based on [this](https://github.com/apache/airflow/pull/36052) made by [VladaZakharova](https://github.com/VladaZakharova). So basically, it contains the entire...