chaokunyang commented on code in PR #2050:
URL: https://github.com/apache/fury/pull/2050#discussion_r1948082427


##########
java/fury-core/src/main/java/org/apache/fury/Fury.java:
##########
@@ -604,6 +604,16 @@ private void writeData(MemoryBuffer buffer, ClassInfo 
classInfo, Object obj) {
       case ClassResolver.STRING_CLASS_ID:
         stringSerializer.writeJavaString(buffer, (String) obj);
         break;
+      case ClassResolver.ARRAYLIST_CLASS_ID:

Review Comment:
   The `STRING_CLASS_ID` and `ARRAYLIST_CLASS_ID` is not continuous, it may 
make the switch clause degenerate  from switch jump, which make the execution 
slower.
   
   Maybe we should remote this fastpath from copyObject too



-- 
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]

Reply via email to