Copilot commented on code in PR #8067:
URL: https://github.com/apache/incubator-seata/pull/8067#discussion_r3173153436
##########
seata-spring-autoconfigure/seata-spring-autoconfigure-client/src/main/java/org/apache/seata/spring/boot/autoconfigure/properties/SeataJsonProperties.java:
##########
@@ -31,13 +31,30 @@
@ConfigurationProperties(prefix = JSON_PREFIX)
public class SeataJsonProperties {
+ /**
+ * JSON serializer implementation to use for Seata JSON
serialization/deserialization.
+ * Supported values include fastjson, fastjson2, jackson, and gson.
Review Comment:
The Javadoc list of supported `serializerType` values is incomplete: the
codebase also supports `jackson3` (see `JsonSerializerFactory`), and the sample
configs in `script/client/spring/*` list `jackson3` as an option. Please update
this comment to include `jackson3` (or reference `JsonSerializerFactory`’s
documented list) to avoid misleading users.
```suggestion
* Supported values include fastjson, fastjson2, jackson, jackson3, and
gson.
```
--
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]
