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


##########
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:
   @DonalEvans that will work but we will still need to add the 
serialVersionUID in that case too so that the already released versions are 
able to operate during the rolling update.
   That effort can be different PR where we add getID to all the missing IDs in 
the InternalFunction



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