+1, keep gson and remove fastjson.
At 2021-02-20 14:15:30, "Yeauty YE" <[email protected]> wrote: > >First of all, dubbo is positioned as a lightweight framework, so in theory, >only one framework needs to be kept for json. Secondly, the scene of json >operation is not the most frequent main scene, so it will not affect dubbo’s >performance. Finally, since it is a framework for internationalization, >security is very important among them. In summary, keep Gson and abandon >FastJson. > >In order to prevent dependency problems caused by user upgrades, I suggest: >* delete FastJSON from `dubbo-all pom.xml` and `dubbo-common pom.xml` (Ensure >the purity of the Dubbo core) >* Fastjson serialization module should be preserved > > >On 2021/02/18 06:43:27, Jun Liu <[email protected]> wrote: >> Hi,> >> >> > For so long, Dubbo has both Gson and FastJSON, and many modules rely on> >> > these two json serialization tools.> >> >> >> I think there’s no need to use two libraries as they provide almost the same >> ability. Personally, I have no inclination about which one to keep. > >> >> > I am worried that removing one will> >> > cause compatibility and stability issues.> >> >> Here’re the possible impacts that I can think of at this moment:> >> * Fastjson serialization module should be preserved> >> * Internal dependency of Dubbo on json can be unified into a library, and >> most scenarios should have no compatibility problems> >> * dubbo-all pom.xml introduces the transmission dependency of both Gson and >> Fastjson. Removing one of them may bring dependency problems to users who >> want to upgrade. Considering that fastjson has just been added in 2.7.8, I >> think we can choose to remove Fastjson for long time maintenance good, Or, >> we can choose to keep both in pom.xml for the time being.> >> >> Jun> >> >> > On Feb 18, 2021, at 12:09 PM, Albumen Kevin <[email protected]> wrote:> >> > > >> > Hi community,> >> > > >> > There is a pull request removing FastJSON dependency for apache/dubbo:> >> > > >> > https://github.com/apache/dubbo/pull/7187> >> > > >> > For so long, Dubbo has both Gson and FastJSON, and many modules rely on> >> > these two json serialization tools. I am worried that removing one will> >> > cause compatibility and stability issues.> >> > > >> > Albumen> >> >>
