hussein-awala commented on PR #36029: URL: https://github.com/apache/airflow/pull/36029#issuecomment-1992548233
@jscheffl The new column is needed to serialize the class instance, to make it possible to create a parameterized strategy, and without this serialization, we cannot re-create the originally provided instance. The goal was to make it generic and flexible to allow users to implement any type of strategy, even if the implemented strategy impacts performance we can simply recommend not to do it but not block it. Anyway, I like @eladkal suggestion of making it experimental, in this case we can implement it in different stages, for example in this release (2.9.0) we can support initiating class without parameters (without the need to be serialized), and in the next release we can decide whether to support the full serialized instance or not. -- 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]
