kirklund commented on a change in pull request #6860:
URL: https://github.com/apache/geode/pull/6860#discussion_r707745200
##########
File path:
geode-junit/src/main/java/org/apache/geode/security/ExpirableSecurityManager.java
##########
@@ -31,44 +32,44 @@
*
* make sure to call reset after each test to clean things up.
*/
-public class ExpirableSecurityManager extends SimpleSecurityManager {
+public class ExpirableSecurityManager extends SimpleSecurityManager implements
Serializable {
Review comment:
This is `Serializable only` because of lambda style VM invocations,
right?
Since this is in the product code and doesn't really need to go on the wire,
you should instead subclass `ExpirableSecurityManager` as an inner-class within
the dunit test and then use that subclass in the tests.
```
private static class SerializableExpirableSecurityManager
extends ExpirableSecurityManager
implements Serializable {
...
}
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
