sodonnel commented on a change in pull request #2554:
URL: https://github.com/apache/ozone/pull/2554#discussion_r695622945
##########
File path:
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/scm/node/TestDecommissionAndMaintenance.java
##########
@@ -98,8 +104,123 @@
private ContainerOperationClient scmClient;
- @Before
- public void setUp() throws Exception {
+ private static MiniClusterProvider clusterProvider;
+
+ /**
+ * Class to create mini-clusters in the background.
+ */
+ public static class MiniClusterProvider {
+
+ private int preCreatedLimit = 1;
+
+ private final OzoneConfiguration conf;
+ private final MiniOzoneCluster.Builder builder;
+ private boolean shouldRun = true;
+ private boolean shouldReap = true;
Review comment:
I think you are correct. I have refactored a bit to remove these
variables as suggested.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
