upthewaterspout commented on a change in pull request #6495:
URL: https://github.com/apache/geode/pull/6495#discussion_r636945594
##########
File path:
geode-core/src/main/java/org/apache/geode/management/internal/configuration/domain/Configuration.java
##########
@@ -169,7 +170,8 @@ public void toData(DataOutput out) throws IOException {
// Before 1.12, this code wrote a non-null HashSet of jarnames to the
output stream.
// As of 1.12, it writes a null HashSet to the stream, so that when we can
still read the old
Review comment:
This comment seems outdated since you are now writing the hash set of
jar names.
--
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]
