gianm commented on a change in pull request #8739: remove select query URL: https://github.com/apache/incubator-druid/pull/8739#discussion_r340346524
########## File path: docs/querying/select-query.md ########## @@ -24,235 +24,11 @@ sidebar_label: "Select" --> -> We encourage you to use the [Scan query](../querying/scan-query.md) type rather than Select whenever possible. -> In situations involving larger numbers of segments, the Select query can have very high memory and performance overhead. +> The native Druid Select query has been removed in favor of the [Scan query](../querying/scan-query.md), which should Review comment: The `>` make this look weird when rendered (a note/quote block). Also the sentences read strangely together. I'd suggest: > Older versions of Apache Druid (incubating) included a Select query type. Since Druid 0.17.0, it has been removed and replaced by the Scan query, which offers improved memory usage and performance. This solves issues that users had with Select queries causing Druid to run out of memory or slow down. > > The Scan query has a different syntax, but supports many of the features of the Select query, including time ordering and limiting. Scan does not include the Select query's pagination feature; however, in many cases pagination is unnecessary with Scan due to its ability to return a virtually unlimited number of results in one call. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
