From 1af90ac2672fd5077758ba68074773e7935c3981 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6ren=20Henning?= <post@soeren-henning.de>
Date: Fri, 31 Jul 2020 12:03:31 +0200
Subject: [PATCH] Minor code clean up
---
.../.settings/org.eclipse.jdt.ui.prefs | 2 +-
.../generators/AbstractWorkloadGenerator.java | 11 +++++++----
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/workload-generator-commons/.settings/org.eclipse.jdt.ui.prefs b/workload-generator-commons/.settings/org.eclipse.jdt.ui.prefs
index 4e04e2891..fa98ca63d 100644
--- a/workload-generator-commons/.settings/org.eclipse.jdt.ui.prefs
+++ b/workload-generator-commons/.settings/org.eclipse.jdt.ui.prefs
@@ -32,7 +32,7 @@ cleanup.qualify_static_member_accesses_with_declaring_class=true
cleanup.qualify_static_method_accesses_with_declaring_class=false
cleanup.remove_private_constructors=true
cleanup.remove_redundant_modifiers=false
-cleanup.remove_redundant_semicolons=false
+cleanup.remove_redundant_semicolons=true
cleanup.remove_redundant_type_arguments=true
cleanup.remove_trailing_whitespaces=true
cleanup.remove_trailing_whitespaces_all=true
diff --git a/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/generators/AbstractWorkloadGenerator.java b/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/generators/AbstractWorkloadGenerator.java
index 82bb5a951..2f2d82346 100644
--- a/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/generators/AbstractWorkloadGenerator.java
+++ b/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/generators/AbstractWorkloadGenerator.java
@@ -87,7 +87,7 @@ public abstract class AbstractWorkloadGenerator<T>
final int periodMs = (int) period.toMillis();
- LOGGER.info("Period: " + periodMs); // NOPMD no computational intensive logger call
+ LOGGER.info("Period: {}", periodMs);
final BiConsumer<WorkloadDefinition, Integer> workerAction = (declaration, workerId) -> {
@@ -118,9 +118,12 @@ public abstract class AbstractWorkloadGenerator<T>
}
};
- this.workloadDistributor =
- new WorkloadDistributor(this.instances, this.zooKeeper, this.keySpace, this.beforeAction,
- workerAction);
+ this.workloadDistributor = new WorkloadDistributor(
+ this.instances,
+ this.zooKeeper,
+ this.keySpace,
+ this.beforeAction,
+ workerAction);
}
/**
--
GitLab