belliottsmith commented on code in PR #4324:
URL: https://github.com/apache/cassandra/pull/4324#discussion_r2315699138


##########
src/java/org/apache/cassandra/utils/ReflectionUtils.java:
##########
@@ -105,4 +105,36 @@ public static <K, V> void clearMapField(Class<?> clazz, 
Object instance, String
             throw new RuntimeException(String.format("Could not clear map 
field %s in class %s", mapName, clazz), ex);
         }
     }
+
+    public static void setFieldToNull(Object object, String fieldName)

Review Comment:
   I think it would be better to accept the declaring class here, so we know 
we're nulling the correct field while also simplifying the implementation.



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