ic4y commented on pull request #1520: URL: https://github.com/apache/arrow-datafusion/pull/1520#issuecomment-1005596623
@alamb Sorry, there are three problems now 1. 3.8.0 version of BumpVec does not implement the Drop trait, so the GroupState is not really freed and there will be a memory leak problem. So the above test results are not accurate. 2. The direct use of BumpVec does not assign the GroupState to bump arena, the allocation is only the GroupState reference 3. On MacOs, there is the problem that drop_in_place takes up a lot of time.However, I tested on linux and didn't find this problem. I’m not familiar with bumpalo yet, but I’m trying to solve these problems. -- 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]
