ccoffline commented on pull request #5332: URL: https://github.com/apache/incubator-doris/pull/5332#issuecomment-771628310
@HangyuanLiu The MySQL documentation is unclear whether the LIMIT clause is in the outer clause or anywhere in the sql that takes over `sql_select_limit`. I would try to figure it out. This commit is an emergency fix for short of memory on FE caused by large result set queries, so I didn't choose a complex implementation. Besides, this is not a variable that would make the app depends on it. The scenario of Doris queries is not exactly the same as MySQL. I think we should figure out in what case the LIMIT clause needs to take over. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
