potiuk commented on PR #38531: URL: https://github.com/apache/airflow/pull/38531#issuecomment-2059261399
> Also regarding the original issue with incorrect serialization for Python 3.11, is it a problem only with serialization or deserialization too? If we can use dill for Python 3.11 only for deserialization, and cloudpickle for serialization, is there will be a problem? The Python 3.11 issue is really a "test" issue - I am not really sure if this has the effect in production. BTW. I think there is a little misunderstanding here. If the only reason for this change is supporting cloudpickle in PVE and not getting rid of dill, then we can likely leave `dill` as mandatory dependency of Airflow and do not worry about the configuration for executor - and only get `cloudpickle` as optional dependency for Python virtualenv operator. Maybe I was assuming too much of a reason for that change when I was commending on dill used in the executor. And we can leave that part altogether - concentrating back on just Python Virtualenv Operator. -- 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]
