[incubator-druid] branch master updated: Emit query metrics even if the ETags are equal (#6663)

[gi...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

gian pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 9a89200  Emit query metrics even if the ETags are equal (#6663)
9a89200 is described below

commit 9a8920060721b68bb2dbd23664fcbf3b925882a7
Author: Mingming Qiu <cs...@gmail.com>
AuthorDate: Wed Nov 28 07:18:01 2018 +0800

    Emit query metrics even if the ETags are equal (#6663)
---
 server/src/main/java/org/apache/druid/server/QueryResource.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/server/src/main/java/org/apache/druid/server/QueryResource.java b/server/src/main/java/org/apache/druid/server/QueryResource.java
index 15b6372..fba6672 100644
--- a/server/src/main/java/org/apache/druid/server/QueryResource.java
+++ b/server/src/main/java/org/apache/druid/server/QueryResource.java
@@ -194,6 +194,8 @@ public class QueryResource implements QueryCountStatsProvider
       final String prevEtag = getPreviousEtag(req);
 
       if (prevEtag != null && prevEtag.equals(responseContext.get(HEADER_ETAG))) {
+        queryLifecycle.emitLogsAndMetrics(null, req.getRemoteAddr(), -1);
+        successfulQueryCount.incrementAndGet();
         return Response.notModified().build();
       }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org