From d1a9b1232fbb1db1dfb481290a183a4c7984af8a Mon Sep 17 00:00:00 2001
From: Sean Busbey <busbey@apache.org>
Date: Sun, 1 Jun 2014 16:19:21 -0500
Subject: [PATCH] [client] clean up status message creation.

closes #237.
---
 core/src/main/java/com/yahoo/ycsb/Client.java | 29 ++++++++-----------
 1 file changed, 12 insertions(+), 17 deletions(-)

diff --git a/core/src/main/java/com/yahoo/ycsb/Client.java b/core/src/main/java/com/yahoo/ycsb/Client.java
index e8b108a4..67aec9fb 100644
--- a/core/src/main/java/com/yahoo/ycsb/Client.java
+++ b/core/src/main/java/com/yahoo/ycsb/Client.java
@@ -102,25 +102,20 @@ class StatusThread extends Thread
 			
 			DecimalFormat d = new DecimalFormat("#.##");
 			String label = _label + format.format(new Date());
-			
-			if (totalops==0)
-			{
-				System.err.println(label+ " " +(interval/1000)+" sec: "+totalops+" operations; "+Measurements.getMeasurements().getSummary());
-			}
-			else
-			{
-				System.err.println(label+" " + (interval/1000)+" sec: "+totalops+" operations; "+d.format(curthroughput)+" current ops/sec; "+Measurements.getMeasurements().getSummary());
-			}
 
-			if (_standardstatus)
-			{
-			if (totalops==0)
-			{
-				System.out.println(label+" "+(interval/1000)+" sec: "+totalops+" operations; "+Measurements.getMeasurements().getSummary());
+			StringBuilder msg = new StringBuilder(label).append(" ").append(interval/1000).append(" sec: ");
+			msg.append(totalops).append(" operations; ");
+
+			if (totalops != 0) {
+				msg.append(d.format(curthroughput)).append(" current ops/sec; ");
 			}
-			else
-			{
-				System.out.println(label+" "+(interval/1000)+" sec: "+totalops+" operations; "+d.format(curthroughput)+" current ops/sec; "+Measurements.getMeasurements().getSummary());			}
+
+			msg.append(Measurements.getMeasurements().getSummary());
+
+			System.err.println(msg);
+
+			if (_standardstatus) {
+				System.out.println(msg);
 			}
 
 			try
-- 
GitLab