zhli1142015 commented on PR #5456: URL: https://github.com/apache/incubator-gluten/pull/5456#issuecomment-2095095918
@zhouyuan and @zhztheplayer , could you help check the memory leak failure? From the log I added and my understaning, this is related to the change https://github.com/apache/incubator-gluten/pull/5558/files#diff-1f31118af1f2bba25c508f4499fbe0304a65f1a681669ab555ce5ba03ed004c4. The reservation is added twice (`pool->grow(0, bytes)` and `pool->grow(bytes, bytes)`) and only one is counted by child pool and can be released. BTW, the memory leask looks not triggered by spilt preloading, but the `facebook::velox::cache::CoalescedLoad::loadOrFuture`.  -- 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]
