mattjubb commented on code in PR #2702:
URL: https://github.com/apache/fory/pull/2702#discussion_r2407469187
##########
java/fory-core/src/main/java/org/apache/fory/resolver/ClassInfoSerializer.java:
##########
@@ -0,0 +1,21 @@
+package org.apache.fory.resolver;
+
+import org.apache.fory.memory.MemoryBuffer;
+
+/**
+ * ClassInfoSerializer provides a mechanism to customize the recording of the
ClassInfo
+ * default implementation uses a short value, however when the class
registration needs to be customized
+ * this can be set of the ClassResolver to handle storing the ClassInfo is a
customized manner
+ */
+public interface ClassInfoSerializer {
Review Comment:
the PR description and the unit test show examples of how this would be
used. Basically we maintain a large mapping of identifiers to classes, when an
object is deserialized we want to control what ID is mapped to what class. I
asked about this on slack a couple months ago, and you said that we cannot
customize the class resolve since it final and you suggest instead to raise a
PR that supports customized mapping logic. the intent of this feature is for it
to be customizable by the calling code not something that is predefined within
the class resolver
--
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]
