clintropolis opened a new pull request #8739: remove select query URL: https://github.com/apache/incubator-druid/pull/8739 ### Description This PR removes the Select query type, in favor of the Scan query. Since the introduction of time ordered scan query with #7133, our documentation has already been recommending people use scan queries instead of select due to the heavy memory impact of the select query, and Druid SQL has not generated select queries at all since #7373. Functionally, scan is equivalent, less the paging functionality, however the more efficient nature of the scan query makes the paging functionality un-necessary. Additionally, I feel that removing the choice between 2 _very_ similar query types should be more user friendly, and for code hygiene and lower maintenance for the query processing layer on our end. This is obviously incompatible with previous releases so will need to be called out prominently in the release notes should we merge this PR. <hr> This PR has: - [x] been self-reviewed. - [x] added documentation for new or modified features or behaviors.
---------------------------------------------------------------- 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]
