DonalEvans commented on code in PR #7529:
URL: https://github.com/apache/geode/pull/7529#discussion_r843044251


##########
geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/GetClusterConfigurationFunction.java:
##########
@@ -27,6 +27,7 @@
 
 public class GetClusterConfigurationFunction implements InternalFunction {
   private static final Logger logger = LogService.getLogger();
+  private static final long serialVersionUID = 6332908511113951823L;

Review Comment:
   I think that implementing `getId()` on the `GetClusterConfigurationFunction` 
class and registering the function using `FunctionService.registerFunction()` 
on each member should allow it to not use Java serialization. I don't know how 
feasible that would be in terms of preserving backwards compatibility though.



-- 
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: notifications-unsubscr...@geode.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to