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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6513113034a refactor: remove id from query/priority metric (#19696)
6513113034a is described below

commit 6513113034a865d78e5b511f9bfa737c91b44997
Author: jtuglu1 <[email protected]>
AuthorDate: Wed Jul 15 15:59:24 2026 -0700

    refactor: remove id from query/priority metric (#19696)
    
    Removes a high-cardinality id dimension that was added in #19665. We should 
re-consider whether we want to keep this change versus add priority dimension 
to the query/time metric at a future time.
---
 server/src/main/java/org/apache/druid/server/QueryScheduler.java | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/server/src/main/java/org/apache/druid/server/QueryScheduler.java 
b/server/src/main/java/org/apache/druid/server/QueryScheduler.java
index 6ec8186c52b..ad43d514890 100644
--- a/server/src/main/java/org/apache/druid/server/QueryScheduler.java
+++ b/server/src/main/java/org/apache/druid/server/QueryScheduler.java
@@ -179,12 +179,6 @@ public class QueryScheduler implements QueryWatcher
                                                                     
.setDimension("lane", lane.orElse(DEFAULT))
                                                                     
.setDimension("dataSource", query.getDataSource().getTableNames())
                                                                     
.setDimension("type", query.getType());
-    // "id" (the query id) lets query/priority be attributed to an individual 
query rather than only aggregated at
-    // datasource/type granularity. High cardinality, so downstream pipelines 
must not forward it to low-cardinality
-    // stores (e.g. Atlas). Guarded because some (typically internal) queries 
carry no id and setDimension rejects null.
-    if (query.getId() != null) {
-      builderUsr.setDimension("id", query.getId());
-    }
     emitter.emit(builderUsr.setMetric("query/priority", 
priority.orElse(Integer.valueOf(0))));
     return lane.map(query::withLane).orElse(query);
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to