andygrove commented on PR #1021: URL: https://github.com/apache/datafusion-comet/pull/1021#issuecomment-2558294675
> I have noticed that for now Comet biases towards always using the unified memory manager, so this patch for native memory management may not be relevant, so I prefer closing this PR. If you still want to merge it into the upstream then I'll take some time to rebase it. Just to add some context here, we have discovered a need to support existing deployments that are not using offheap memory, so would like to add a non-offheap option back in, and this PR seems like the best starting point for deciding how we want to configure the memory pools in that case. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
