Change in asterixdb[master]: Report integer metrics as numbers

[Till Westmann (Code Review)]

Till Westmann has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1496

Change subject: Report integer metrics as numbers
......................................................................

Report integer metrics as numbers

Change-Id: Ie330ba0e030d188adbf675741c382cf2754853f1
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
1 file changed, 11 insertions(+), 4 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/96/1496/1

diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
index 744b929..7cb492b 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
@@ -337,11 +337,18 @@
     }
 
     private static void printField(PrintWriter pw, String name, String value, 
boolean comma) {
+        printFieldInternal(pw, name, "\"" + value + "\"", comma);
+    }
+
+    private static void printField(PrintWriter pw, String name, long value, 
boolean comma) {
+        printFieldInternal(pw, name, String.valueOf(value), comma);
+    }
+
+    private static void printFieldInternal(PrintWriter pw, String name, String 
value, boolean comma) {
         pw.print("\t\"");
         pw.print(name);
-        pw.print("\": \"");
+        pw.print("\": ");
         pw.print(value);
-        pw.print('"');
         if (comma) {
             pw.print(',');
         }
@@ -409,9 +416,9 @@
         pw.print("\t");
         printField(pw, Metrics.EXECUTION_TIME.str(), 
TimeUnit.formatNanos(executionTime));
         pw.print("\t");
-        printField(pw, Metrics.RESULT_COUNT.str(), 
String.valueOf(resultCount));
+        printField(pw, Metrics.RESULT_COUNT.str(), resultCount, true);
         pw.print("\t");
-        printField(pw, Metrics.RESULT_SIZE.str(), String.valueOf(resultSize), 
false);
+        printField(pw, Metrics.RESULT_SIZE.str(), resultSize, false);
         pw.print("\t}\n");
     }
 

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1496
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie330ba0e030d188adbf675741c382cf2754853f1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <ti...@apache.org>