vldpyatkov commented on a change in pull request #8113:
URL: https://github.com/apache/ignite/pull/8113#discussion_r466564947



##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/binary/BinaryUtils.java
##########
@@ -1998,6 +2027,14 @@ public static Object doReadOptimized(BinaryInputStream 
in, BinaryContext ctx, @N
         }
     }
 
+    /**
+     * @param ctx Context.
+     * @param clsLdr Class loader.
+     */
+    public static boolean isClassCacheWillUsed(BinaryContext ctx, ClassLoader 
clsLdr) {

Review comment:
       This method looks suspicion, moreover we have 
GridBinaryMarshaller.USE_CACHE.
   Why we cannot use the thread local variable everywhere?




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to