ChahatKumar commented on code in PR #73783: URL: https://github.com/apache/airflow/pull/73783#discussion_r4131216500
########## task-sdk/src/airflow/sdk/execution_time/supervisor.py: ########## Review Comment: Thanks for fixing the large-integer MessagePack failure in this PR. We found one remaining Airflow 2/3 difference: a DagRun.conf value such as 1.7E308 is a Python float in Airflow 2, but becomes a 309-digit Python int in Airflow 3 after the PostgreSQL JSONB round trip. The initial Airflow 3 API parsing does preserve it as a float; the type changes during persistence. Is this expected, or should Airflow 3 preserve the original numeric type across DagRun.conf persistence @dhkim1920 @rozza @imbaczek -- 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]
