boneanxs commented on issue #12911: URL: https://github.com/apache/gluten/issues/12911#issuecomment-5447647594
> 12729 is already picked into gluten. > > Is it reproducible? Yes, it is reproducible, I mean #12729 might cause this issue. Our instrumentation shows that the async load submitted by DirectBufferedInput::makeLoads() can still be queued after Gluten's 30s runtime teardown timeout; it has not started running. The AsyncLoadHolder introduced by #12729 intentionally retains a shared_ptr<MemoryPool> while that queued closure exists, so the task root pool remains alive (refcount 2) even though its usage is 0. Gluten then proceeds to destroy NativeMemoryManager after the timeout, and Velox reports pools_.size() != 0. -- 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]
