You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by fj...@apache.org on 2018/09/25 00:56:44 UTC

[incubator-druid] branch master updated: Deprecate dimensions / metrics APIs on brokers (#6361)

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

fjy 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 99428e2  Deprecate dimensions / metrics APIs on brokers (#6361)
99428e2 is described below

commit 99428e20d210bb70cd28fe146ccb572fd036bafc
Author: Jihoon Son <ji...@apache.org>
AuthorDate: Mon Sep 24 17:56:38 2018 -0700

    Deprecate dimensions / metrics APIs on brokers (#6361)
    
    * Deprecate dimensions / metrics APIs on brokers
    
    * add segmentMetadataQuery link
    
    * add more doc
---
 docs/content/operations/api-reference.md                     | 12 ++++++++++++
 .../java/org/apache/druid/server/ClientInfoResource.java     |  2 ++
 2 files changed, 14 insertions(+)

diff --git a/docs/content/operations/api-reference.md b/docs/content/operations/api-reference.md
index 2a58de0..cacab7e 100644
--- a/docs/content/operations/api-reference.md
+++ b/docs/content/operations/api-reference.md
@@ -425,10 +425,22 @@ druid.query.segmentMetadata.defaultHistory
 
 Returns the dimensions of the datasource.
 
+<div class="note caution">
+This API is deprecated and will be removed in future releases. Please use [SegmentMetadataQuery](../querying/segmentmetadataquery.html) instead
+which provides more comprehensive information and supports all dataSource types including streaming dataSources. It's also encouraged to use [INFORMATION_SCHEMA tables](../querying/sql.html#retrieving-metadata)
+if you're using SQL.
+</div>
+
 * `/druid/v2/datasources/{dataSourceName}/metrics`
 
 Returns the metrics of the datasource.
 
+<div class="note caution">
+This API is deprecated and will be removed in future releases. Please use [SegmentMetadataQuery](../querying/segmentmetadataquery.html) instead
+which provides more comprehensive information and supports all dataSource types including streaming dataSources. It's also encouraged to use [INFORMATION_SCHEMA tables](../querying/sql.html#retrieving-metadata)
+if you're using SQL.
+</div>
+
 * `/druid/v2/datasources/{dataSourceName}/candidates?intervals={comma-separated-intervals-in-ISO8601-format}&numCandidates={numCandidates}`
 
 Returns segment information lists including server locations for the given datasource and intervals. If "numCandidates" is not specified, it will return all servers for each interval.
diff --git a/server/src/main/java/org/apache/druid/server/ClientInfoResource.java b/server/src/main/java/org/apache/druid/server/ClientInfoResource.java
index f18799e..b60cb8d 100644
--- a/server/src/main/java/org/apache/druid/server/ClientInfoResource.java
+++ b/server/src/main/java/org/apache/druid/server/ClientInfoResource.java
@@ -220,6 +220,7 @@ public class ClientInfoResource
     return result;
   }
 
+  @Deprecated
   @GET
   @Path("/{dataSourceName}/dimensions")
   @Produces(MediaType.APPLICATION_JSON)
@@ -253,6 +254,7 @@ public class ClientInfoResource
     return dims;
   }
 
+  @Deprecated
   @GET
   @Path("/{dataSourceName}/metrics")
   @Produces(MediaType.APPLICATION_JSON)


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