chaokunyang commented on code in PR #2837:
URL: https://github.com/apache/fory/pull/2837#discussion_r2488553305
##########
README.md:
##########
@@ -222,6 +222,7 @@ public class Example {
// replace `build` with `buildThreadSafeFory` for Thread-Safe Usage
.build();
// Register your classes (required when class registration is enabled)
+ // NOTE: The registration order is important. If registration order is
inconsistent, the allocated ID will be different, and the deserialization will
failed !!!
Review Comment:
The order only ,atters if not registered with id/name, could we clarify that
and simplify comments here? I prefer keeping main readme consise
--
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]
