Shahar Epstein

Results 59 comments of Shahar Epstein

I'm reopening this issue as the original PR (#34337) that should have solved it was [reverted](https://github.com/astronomer/airflow/commit/b4321bb0947cec5c8ba17382f8fe1dfc3f8cfed4), and no other fix has been merged since then. I'd like to take a...

I've resolved the issue for version 2.X, so hopefully it will be available starting v2.10.5/v2.11.0 (whatever comes first). I leave this issue open as there are some arrangements to do...

> @shahar1 sir could you please advise on the appropriate folder for placing unit test cases? `tests/providers/amazon/aws/operators/`, and there check all the files that start with `emr_`.

> @shahar1 sir and @vincbeck sir I've added unit test cases and done my best to cover the expected behaviors. As I'm a beginner, I'd greatly appreciate your feedback and...

> @shahar1 sir and @vincbeck sir I've tried to resolve conflict and done my best to cover the expected behaviors. As I'm a beginner, I'd greatly appreciate your feedback and...

Hey @Chais, I've managed to reproduce the issue using the code snippets that you provided. ~The root cause of this issue is that short circuit isn't currently suitable for working...

> Shouldn’t `downstream_task_ids` still contain the unexpanded base task in this case? Something still seems off to me. And we should probably fix the operator in any case. After spending...

> Why does not recording TaskMap affect this? The function only does one thing—pushing a row to the TaskMap table. This table is only used for task mapping. You're correct,...

I've resolved the issue for version 2.X, so hopefully it will be available starting v2.10.5/v2.11.0 (whatever comes first). I leave this issue open as there are some arrangements to do...