MaksYermak commented on PR #55778: URL: https://github.com/apache/airflow/pull/55778#issuecomment-3311526461
> Generally you should target main with any bug fixes, unless there is a specific reason why it only applies to the 2.x branch. @ashb as I mentioned in PR's description this problem doesn't exist in AF3(`main` branch). And it is a reason why I prepared this fix for `2.11` only. > This problem doesn't exist for AF3. Because for AF3 the code [raise](https://github.com/apache/airflow/blob/main/task-sdk/src/airflow/sdk/definitions/param.py#L68C1-L72C1) exception and failed immediately in place where discover the non-json-serializable data. In the AF2 the code [shows](https://github.com/apache/airflow/blob/v2-11-test/airflow/models/param.py#L74C1-L82C1) the warning that using non-json-serializable parameters will be deprecated in AF3. It does not prevent airflow from crashing later with more dramatic consequences. -- 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]
