gesterzhou commented on code in PR #7594:
URL: https://github.com/apache/geode/pull/7594#discussion_r851528925
##########
geode-assembly/geode-assembly-test/src/main/java/org/apache/geode/session/tests/ContainerInstall.java:
##########
@@ -248,32 +252,36 @@ int getDefaultLocatorPort() {
return defaultLocatorPort;
}
- /**
+ /*
* Gets the cache XML file to use by default for this installation
*/
File getCacheXMLFile() {
return new File(modulePath + "/conf/" +
getConnectionType().getCacheXMLFileName());
}
- /**
+ /*
* Cargo specific string to identify the container with
*/
public abstract String getInstallId();
- /**
+ /*
* A human readable description of the installation
*/
public abstract String getInstallDescription();
- /**
+ /*
* Get the session manager class to use
*/
public abstract String getContextSessionManagerClass();
/**
* Generates a {@link ServerContainer} from the given {@link
ContainerInstall}
*
+ * @param containerConfigHome The folder that the container configuration
folder should be setup
Review Comment:
these 2 lines can be adjusted
--
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
