From 1b1fd27e3fbda8b9ebb2aa9f203cca69b6b30983 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6ren=20Henning?= <soeren.henning@email.uni-kiel.de>
Date: Wed, 24 Feb 2021 17:51:18 +0100
Subject: [PATCH] Add JavaDoc

---
 .../workloadgeneration/HazelcastRunner.java       |  9 +++++++++
 .../HazelcastRunnerStateInstance.java             | 15 ++++++++++++---
 2 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunner.java b/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunner.java
index a2a9c1411..72eccc66a 100644
--- a/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunner.java
+++ b/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunner.java
@@ -10,6 +10,9 @@ import com.hazelcast.core.HazelcastInstance;
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 
+/**
+ * A Theodolite load generator runner that establishes a cluster using Hazelcast.
+ */
 public class HazelcastRunner {
 
   private final HazelcastInstance hzInstance;
@@ -18,6 +21,9 @@ public class HazelcastRunner {
   private final LoadGeneratorConfig loadConfig;
   private final WorkloadDefinition totalLoadDefinition;
 
+  /**
+   * Create a new {@link HazelcastRunner} from the given configuration.
+   */
   public HazelcastRunner(
       final ClusterConfig clusterConfig,
       final LoadGeneratorConfig loadConfig,
@@ -28,6 +34,9 @@ public class HazelcastRunner {
     this.hzInstance.getCluster().addMembershipListener(new RunnerMembershipListener());
   }
 
+  /**
+   * Start the workload generation and blocks until the workload generation is stopped again.
+   */
   public void runBlocking() {
     while (!this.stopAction.isDone()) {
       synchronized (this) {
diff --git a/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunnerStateInstance.java b/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunnerStateInstance.java
index 6830cf793..d8fd7de42 100644
--- a/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunnerStateInstance.java
+++ b/benchmarks/workload-generator-commons/src/main/java/theodolite/commons/workloadgeneration/HazelcastRunnerStateInstance.java
@@ -15,7 +15,10 @@ import java.util.stream.Collectors;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-// TODO Reconsider name -> Hazelcast Instance
+/**
+ * An instance of a Hazelcast runner state, that is a load generator cluster with a given set of
+ * members.
+ */
 public class HazelcastRunnerStateInstance {
 
   private static final Logger LOGGER = LoggerFactory.getLogger(HazelcastRunnerStateInstance.class);
@@ -31,6 +34,9 @@ public class HazelcastRunnerStateInstance {
   private final HazelcastInstance hzInstance;
   private final Set<Member> members;
 
+  /**
+   * Create a new {@link HazelcastRunnerStateInstance}.
+   */
   public HazelcastRunnerStateInstance(
       final LoadGeneratorConfig loadGeneratorConfig,
       final WorkloadDefinition totalLoadDefinition,
@@ -44,6 +50,9 @@ public class HazelcastRunnerStateInstance {
     LOGGER.info("Created new Hazelcast runner instance for member set '{}'", this.members);
   }
 
+  /**
+   * Start and block load generation for the configured member set.
+   */
   public void runBlocking() {
     if (!this.stopAction.isDone()) {
       this.tryPerformBeforeAction();
@@ -175,8 +184,8 @@ public class HazelcastRunnerStateInstance {
   }
 
   private static final class LoadDefPerMember {
-    public final WorkloadDefinition loadDefinition;
-    public final Member member;
+    public final WorkloadDefinition loadDefinition; // NOCS used only internally
+    public final Member member; // NOCS used only internally
 
     public LoadDefPerMember(final WorkloadDefinition loadDefinition, final Member member) {
       this.loadDefinition = loadDefinition;
-- 
GitLab