kevinhongzl commented on PR #52422: URL: https://github.com/apache/airflow/pull/52422#issuecomment-3016790866
Hi @potiuk @kaxil. I might need your help on yandex backward compat tests. Previously, when we called `persist`, the function call chain was `persist()` -> `BaseOperator.push_xcom()` -> `TI.push_xcom()`. A redundant arg (`execution_date`) was added during the BashOperator Xcom push. However, in our current `persist()`, we push Xcoms directly through a task instance without adding `execution_date`. This means the rewrite will change how `persist()` behaves in AF2 and its outputs will no longer match the expected output of the original test cases. Given this, should I just modify the AF2 test cases, or is there a better way to handle this? Thanks. -- 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]
