nabarunnag commented on code in PR #7529:
URL: https://github.com/apache/geode/pull/7529#discussion_r843309799
##########
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 it will be tough, I think this needs to be solved as a larger
scale. As you can see that the Serializable comes from InternalFunction class
and we have all ton of these internal functions that are needed for Geode
operation.
@pivotal-jbarrett I went with the status quo with this solution, the
function in the already released older versions were using Java Serialization
hence I kept the solution in the same serialization domain. If there is was to
use two different type of serialization for the class in different versions.
--
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
