Code review comment: Consider moving the comment (or comma) here and above to make the inline comment point at the right parameter

Code before review:
protected void setUp(boolean dontInitializeNode2) throws Exception {
Distribution distribution = new Distribution(Distribution.getSimpleGroupConfig(2, 10));
jsonWriter.setDefaultPathPrefix("/cluster/v2");
{
Set<ConfiguredNode> nodes = FleetControllerTest.toNodes(0, 1, 2, 3);
ContentCluster cluster = new ContentCluster(
"books", nodes, distribution, 6 /* minStorageNodesUp*/, 0.9, /* minRatioOfStorageNodesUp */true);
initializeCluster(cluster, nodes);
AnnotatedClusterState baselineState = AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:4 storage:4"));
Map<String, AnnotatedClusterState> bucketSpaceStates = new HashMap<>();
bucketSpaceStates.put("default", AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:4 storage:4 .3.s:m")));
bucketSpaceStates.put("global", baselineState);
books = new ClusterControllerMock(cluster, baselineState.getClusterState(),
ClusterStateBundle.of(baselineState, bucketSpaceStates), 0, 0);
}
{
Set<ConfiguredNode> nodes = FleetControllerTest.toNodes(1, 2, 3, 5, 7);
Set<ConfiguredNode> nodesInSlobrok = FleetControllerTest.toNodes(1, 3, 5, 7);
ContentCluster cluster = new ContentCluster(
"music", nodes, distribution, 4 /* minStorageNodesUp*/, 0.0, /* minRatioOfStorageNodesUp */true);
if (dontInitializeNode2) {
initializeCluster(cluster, nodesInSlobrok);
}
else {
initializeCluster(cluster, nodes);
}
AnnotatedClusterState baselineState = AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:8 .0.s:d .2.s:d .4.s:d .6.s:d "
+ "storage:8 .0.s:d .2.s:d .4.s:d .6.s:d"));
music = new ClusterControllerMock(cluster, baselineState.getClusterState(),
ClusterStateBundle.ofBaselineOnly(baselineState), 0, 0);
}
ccSockets = new TreeMap<>();
ccSockets.put(0, new ClusterControllerStateRestAPI.Socket("localhost", 80));
restAPI = new ClusterControllerStateRestAPI(new ClusterControllerStateRestAPI.FleetControllerResolver() {
@Override
public Map<String, RemoteClusterControllerTaskScheduler> getFleetControllers() {
Map<String, RemoteClusterControllerTaskScheduler> fleetControllers = new LinkedHashMap<>();
fleetControllers.put(books.context.cluster.getName(), books);
fleetControllers.put(music.context.cluster.getName(), music);
return fleetControllers;
}
}, ccSockets);
}

Line under review: "music", nodes, distribution, 4 /* minStorageNodesUp*/, 0.0, /* minRatioOfStorageNodesUp */true);

Code after addressing the review:
protected void setUp(boolean dontInitializeNode2) throws Exception {
Distribution distribution = new Distribution(Distribution.getSimpleGroupConfig(2, 10));
jsonWriter.setDefaultPathPrefix("/cluster/v2");
{
Set<ConfiguredNode> nodes = FleetControllerTest.toNodes(0, 1, 2, 3);
ContentCluster cluster = new ContentCluster(
"books", nodes, distribution, 6 /* minStorageNodesUp*/, 0.9 /* minRatioOfStorageNodesUp */,
true /* determineBucketsFromBucketSpaceMetric */);
initializeCluster(cluster, nodes);
AnnotatedClusterState baselineState = AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:4 storage:4"));
Map<String, AnnotatedClusterState> bucketSpaceStates = new HashMap<>();
bucketSpaceStates.put("default", AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:4 storage:4 .3.s:m")));
bucketSpaceStates.put("global", baselineState);
books = new ClusterControllerMock(cluster, baselineState.getClusterState(),
ClusterStateBundle.of(baselineState, bucketSpaceStates), 0, 0);
}
{
Set<ConfiguredNode> nodes = FleetControllerTest.toNodes(1, 2, 3, 5, 7);
Set<ConfiguredNode> nodesInSlobrok = FleetControllerTest.toNodes(1, 3, 5, 7);
ContentCluster cluster = new ContentCluster(
"music", nodes, distribution, 4 /* minStorageNodesUp*/, 0.0 /* minRatioOfStorageNodesUp */,
true /* determineBucketsFromBucketSpaceMetric */);
if (dontInitializeNode2) {
initializeCluster(cluster, nodesInSlobrok);
}
else {
initializeCluster(cluster, nodes);
}
AnnotatedClusterState baselineState = AnnotatedClusterState.withoutAnnotations(ClusterState.stateFromString("distributor:8 .0.s:d .2.s:d .4.s:d .6.s:d "
+ "storage:8 .0.s:d .2.s:d .4.s:d .6.s:d"));
music = new ClusterControllerMock(cluster, baselineState.getClusterState(),
ClusterStateBundle.ofBaselineOnly(baselineState), 0, 0);
}
ccSockets = new TreeMap<>();
ccSockets.put(0, new ClusterControllerStateRestAPI.Socket("localhost", 80));
restAPI = new ClusterControllerStateRestAPI(new ClusterControllerStateRestAPI.FleetControllerResolver() {
@Override
public Map<String, RemoteClusterControllerTaskScheduler> getFleetControllers() {
Map<String, RemoteClusterControllerTaskScheduler> fleetControllers = new LinkedHashMap<>();
fleetControllers.put(books.context.cluster.getName(), books);
fleetControllers.put(music.context.cluster.getName(), music);
return fleetControllers;
}
}, ccSockets);
}