Rename and use correct time unit

Author: Xin Dong

src/Constants.cpp

@@ -76,6 +76,6 @@ const char* MT_TIME_INSERT_LATENCY_US = "dl_insert_latency_useconds";
 const char* MT_HIST_INSERT_SZ = "dl_insert_size_bytes";
 const char* MT_HIST_TR_PER_REQUEST = "dl_tr_per_request";
 const char* MT_RATE_IDX_REBUILD = "dl_index_rebuild_rate";
-const uint64_t SLOW_QUERY_THRESHOLD_SECONDS = 10;
+const uint64_t SLOW_QUERY_THRESHOLD_MICRO_SECONDS = 10000;
 
 } // namespace DocLayerConstants

src/Constants.h

@@ -90,7 +90,7 @@ extern const char* MT_HIST_INSERT_SZ;
 extern const char* MT_HIST_TR_PER_REQUEST;
 extern const char* MT_RATE_IDX_REBUILD;
 // TODO: make this part of the knob/options instead of constants?
-extern const uint64_t SLOW_QUERY_THRESHOLD_SECONDS;
+extern const uint64_t SLOW_QUERY_THRESHOLD_MICRO_SECONDS;
 
 } // namespace DocLayerConstants
 

src/ExtMsg.actor.cpp

@@ -514,13 +514,13 @@ ACTOR static Future<Void> doRun(Reference<ExtMsgQuery> query, Reference<ExtConne
 		}
 	}
 
-	uint64_t queryLatencySeconds = (timer_int() - startTime) / 1000;
-	DocumentLayer::metricReporter->captureTime(DocLayerConstants::MT_TIME_QUERY_LATENCY_US, queryLatencySeconds);
+	uint64_t queryLatencyMicroSeconds = (timer_int() - startTime) / 1000;
+	DocumentLayer::metricReporter->captureTime(DocLayerConstants::MT_TIME_QUERY_LATENCY_US, queryLatencyMicroSeconds);
 
-	if (slowQueryLogging && queryLatencySeconds >= DocLayerConstants::SLOW_QUERY_THRESHOLD_SECONDS) {
-		TraceEvent("SlowQuery")
-		    .detail("Threshold", DocLayerConstants::SLOW_QUERY_THRESHOLD_SECONDS)
-		    .detail("Duration", queryLatencySeconds)
+	if (slowQueryLogging && queryLatencyMicroSeconds >= DocLayerConstants::SLOW_QUERY_THRESHOLD_MICRO_SECONDS) {
+		TraceEvent(SevWarn, "SlowQuery")
+		    .detail("ThresholdMicroSeconds", DocLayerConstants::SLOW_QUERY_THRESHOLD_MICRO_SECONDS)
+		    .detail("DurationMicroSeconds", queryLatencyMicroSeconds)
 		    .detail("Query", query->toString());
 	}