You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/08/04 21:37:42 UTC

[GitHub] [druid] gianm opened a new pull request #11548: Docs: add pointers from api-reference to sql docs.

gianm opened a new pull request #11548:
URL: https://github.com/apache/druid/pull/11548


   Giving people a little push towards using system tables when they can. (They're easier to use than the JSON APIs and more flexible)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [druid] suneet-s commented on a change in pull request #11548: Docs: add pointers from api-reference to sql docs.

Posted by GitBox <gi...@apache.org>.
suneet-s commented on a change in pull request #11548:
URL: https://github.com/apache/druid/pull/11548#discussion_r685257815



##########
File path: docs/operations/api-reference.md
##########
@@ -927,6 +935,11 @@ Returns segment information lists including server locations for the given query
 
 #### GET
 
+> Note: Much of this information is available in a simpler, easier-to-use form through the Druid SQL
+> [`INFORMATION_SCHEMA.TABLES`](../querying/sql.md#tables-table),
+> [`INFORMATION_SCHEMA.COLUMNS`](../querying/sql.md#columns-table), and
+> [`sys.segments`](../querying/sql.md#segments-table) tables.

Review comment:
       nit: Is there a reason why one would choose these APIs over what's available via Druid SQL or vice versa.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [druid] gianm merged pull request #11548: Docs: add pointers from api-reference to sql docs.

Posted by GitBox <gi...@apache.org>.
gianm merged pull request #11548:
URL: https://github.com/apache/druid/pull/11548


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [druid] gianm commented on a change in pull request #11548: Docs: add pointers from api-reference to sql docs.

Posted by GitBox <gi...@apache.org>.
gianm commented on a change in pull request #11548:
URL: https://github.com/apache/druid/pull/11548#discussion_r686192131



##########
File path: docs/operations/api-reference.md
##########
@@ -927,6 +935,11 @@ Returns segment information lists including server locations for the given query
 
 #### GET
 
+> Note: Much of this information is available in a simpler, easier-to-use form through the Druid SQL
+> [`INFORMATION_SCHEMA.TABLES`](../querying/sql.md#tables-table),
+> [`INFORMATION_SCHEMA.COLUMNS`](../querying/sql.md#columns-table), and
+> [`sys.segments`](../querying/sql.md#segments-table) tables.

Review comment:
       The reason lies in the words "Much of this information". Some info is only available in the JSON APIs. (I didn't go through and enumerate which specific info that is, though.)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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