Merge pull request #110 from dabaitu/addCpuTime

Add query cpu time for BigQuery comparison
twitter-forks · Oct 17, 2017 · 16b8aa6 · 16b8aa6
2 parents 8b0e19e + b7a4ed1
commit 16b8aa6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/twitter-eventlistener-plugin/pom.xml b/twitter-eventlistener-plugin/pom.xml
@@ -45,7 +45,7 @@
         <dependency>
             <groupId>com.twitter</groupId>
             <artifactId>presto-thrift-java</artifactId>
-            <version>0.0.4</version>
+            <version>0.0.5</version>
             <exclusions>
                 <exclusion>
                     <groupId>com.twitter</groupId>

diff --git a/...gin/src/main/java/com/twitter/presto/plugin/eventlistener/QueryCompletedEventScriber.java b/...gin/src/main/java/com/twitter/presto/plugin/eventlistener/QueryCompletedEventScriber.java
@@ -88,6 +88,7 @@ private static QueryCompletionEvent toThriftQueryCompletionEvent(QueryCompletedE
     thriftEvent.query_wall_time_ms = eventStat.getWallTime().toMillis();
     thriftEvent.cumulative_memory_bytesecond = eventStat.getCumulativeMemory();
     thriftEvent.peak_memory_bytes = eventStat.getPeakMemoryBytes();
+    thriftEvent.cpu_time_ms = eventStat.getCpuTime().toMillis();
     if (eventStat.getAnalysisTime().isPresent()) {
       thriftEvent.analysis_time_ms = eventStat.getAnalysisTime().get().toMillis();
     }