[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-12 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1096626367 Lets try to address this later. it's not really part of the deprecation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-12 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1096450143 I think https://github.com/apache/airflow/pull/22832/commits/a45380d5d9b3884c8d54dd0f029691ddff94829d should address it. BTW user can do: ``` class MyDummyOperator(DummyOpe

[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-12 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1096391277 yes you are right. Working on it -- 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

[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-12 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1096212901 > There are also a couple of DummyOperator usages in tests we should change. yep there are tests, example dags and other that i need to change. I will do that in a followup PR as

[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-11 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1096065814 hi @ashb @uranusjr WDYT? I'd like to get it in 2.3.0 if possible -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] eladkal commented on pull request #22832: Deprecate `DummyOperator` in favor of `EmptyOperator`

2022-04-08 Thread GitBox
eladkal commented on PR #22832: URL: https://github.com/apache/airflow/pull/22832#issuecomment-1093014502 We have a green build 🟢 it took a while to solve the failure in provider packages wheel test https://github.com/apache/airflow/pull/22832/commits/7a3ed38ee5fc3467844e5d26975420cce9a2