Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/126#discussion_r142683487
--- Diff:
qa/src/test/java/org/apache/brooklyn/qa/load/AbstractLoadTest.java ---
@@ -161,8 +161,8 @@ protected ManagementContext setUpPlatform() {
// Create management node
persistenceDir = Files.createTempDir();
launcher = BrooklynLauncher.newInstance()
- .persistMode(PersistMode.CLEAN)
- .highAvailabilityMode(HighAvailabilityMode.MASTER)
+ .persistMode(doPersistence() ? PersistMode.CLEAN :
PersistMode.DISABLED)
+ .highAvailabilityMode(doPersistence() ?
HighAvailabilityMode.MASTER : HighAvailabilityMode.DISABLED)
--- End diff --
I think for a flag, a field would be cleaner than a method to override
---
