suneet-s commented on a change in pull request #9704: Refresh query docs. URL: https://github.com/apache/druid/pull/9704#discussion_r408992688
########## File path: docs/querying/datasource.md ########## @@ -22,43 +22,317 @@ title: "Datasources" ~ under the License. --> Review comment: Overall comment: All the SQL queries don't have time filters. As someone new to Druid, I'd look at this page and start issuing SQL commands without time filters. We should point out that this will cause performance issues on anything other than small test data. Don't know what the best way to do 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org