EmilyMatt opened a new issue, #1661: URL: https://github.com/apache/datafusion-comet/issues/1661
### What is the problem the feature request solves? As there is now more than one memory Comet pool, and it stands to reason there will be more in the future, as well as with the increasing complexity of the configuration of the pools (as seen with the recent PRs regarding tracked pools etc), I think it may be beneficial to start separating the memory pools into their own module in the code. ### Describe the potential solution A dedicated "memory_pools" directory, with a file for each pool type, as well as a separate file for the configuration. ### Additional context This will also clear up the jni_api.rs file, which is already a bit laden with code, I think. -- 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.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
