null check postPhase in Worker.run to resolve race condition at t…

…erminate (cmu-db#211)

src/main/java/com/oltpbenchmark/api/Worker.java

@@ -293,6 +293,14 @@ public final void run() {
                         // that either started during the warmup phase or ended
                         // after the timer went off.
                         Phase postPhase = workloadState.getCurrentPhase();
+
+                        if (postPhase == null) {
+                            // Need a null check on postPhase since current phase being null is used in WorkloadState
+                            // and ThreadBench as the indication that the benchmark is over. However, there's a race
+                            // condition with postState not being changed from MEASURE to DONE yet, so we entered the
+                            // switch. In this scenario, just break from the switch.
+                            break;
+                        }
                         if (preState == MEASURE && postPhase.getId() == prePhase.getId()) {
                             latencies.addLatency(transactionType.getId(), start, end, this.id, prePhase.getId());
                             intervalRequests.incrementAndGet();