xinyiZzz commented on pull request #8322: URL: https://github.com/apache/incubator-doris/pull/8322#issuecomment-1072373958
> @xinyiZzz When I test for #8451 , I encounter a memory limit problem. > > The problem is that, after the long query, as specified in the test steps of #8451 , is finished, a simple query 'select count() from tableA' will raise memory limit error. > > I guess it's related to this pr, since the problem is not present before I merge the new MemTracker code. @xiaokang Sorry I just saw, The memory should be independent between queries, I will try to find the cause of this problem. Another problem, after this pr, there may be cases where large queries that were successful before, now fail, because this pr implements Query-level memory limits. exec_mem_limit is used to limit the memory of the entire query, which is its original meaning. And in the past, exec_mem_limit is actually the memory limit at the Fragment Instance level, not the query. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
