potiuk commented on issue #50752:
URL: https://github.com/apache/airflow/issues/50752#issuecomment-2920550541
I think both approaches are fine.
2. means pendulum2 is not supported when pendulum 3 is used by Airflow
1. means that we add new feature to also support Pendulum in the venv.
If somoene (@laksh-krishna-sharma wants to work on it and someone will help
to provide guidance 1. is viable).
Guidance: Python virtualenv operator performs serialization of parameters
and then there is a script executed to deserialize it - likely both sides
(serializing and deserializing) should be updated to handle the case of
Pendulum 3 -> 2 serialization.
--
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]
