pitrou commented on PR #38116: URL: https://github.com/apache/arrow/pull/38116#issuecomment-1771389464
Please let's stop talking about design patterns here, as that only obscures the discussion. We have two possible solutions: 1) a class with only static members, and global state 2) an instantiable class, with per-instance state, and also a global default instance Solution 2 is stricly more flexible and more future-proof than solution 1, so why are you insisting on solution 1? Also, why would it be easier to call `MemoryPool::some_method` than `default_memory_pool()->some_method()`, for example? -- 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]
