bolkedebruin commented on issue #50867: URL: https://github.com/apache/airflow/issues/50867#issuecomment-2897238309
Serializers / Deserializers are in https://github.com/apache/airflow/tree/main/airflow-core/src/airflow/serialization/serializers , you can take those as an example. You would need either need to extend`EmbedByTypeResponseEmbeddings` with two methods (serialize, deserialize) but that requires a new object or write a custom serializer per the above examples. The fast way is to add it to the above set of serializers, the proper way would be to extend the provider schema with "serializers" and put it in the provider. I realize that this might be a bit much for a first PR. If it is put in the provider it can be made backwards compatible. If put in core I would consider it a feature that does not warrant a backport. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
