tatiana edited a comment on pull request #9847: URL: https://github.com/apache/airflow/pull/9847#issuecomment-661829786
Thank you both for taking the time to review and give feedback. @kaxil as mentioned by @turbaszek, it would be great if we could avoid introducing this cyclic dependency. I fully agree with avoiding duplicating the code as well. How would you both feel if I try to extract the `BaseSerialization` - keeping its current interfaces - to another module, which would be used both by `xcom.py` and `serialized_objects.py`? If you like this possibility, how would you like name this file which would contain the `BaseSerialization` class? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
