YoWuwuuuw commented on PR #7639: URL: https://github.com/apache/incubator-seata/pull/7639#issuecomment-3298900387
> > I think you misunderstood the purpose of this issue. We need to unify Jackson and FastJSON—the JSON serialization libraries—under a single configuration option and provide a factory that returns the corresponding abstract implementation based on that configuration. First, define an interface that abstracts the JSON-related functionality Seata depends on. Then implement that interface twice: once using FastJSON and once using Jackson. Finally, replace all JSON serialization/deserialization code paths to go through a single JSON utility class; this utility class should perform serialization and deserialization and internally load the selected implementation via SPI according to the configuration. > > Thanks for the detailed guidance! This is a much better approach. I now understand the plan is to create a common interface and use SPI to load different JSON implementations based on configuration. I will start working on it. I'll let you know if I have any questions I think you can refer to the 'discovery' module. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
