zanmato1984 commented on PR #45210: URL: https://github.com/apache/arrow/pull/45210#issuecomment-2581076998
Hi @kou , I've introduced a group of `_allocator` "mix-in"s. These mix-ins are not to be used as regular inheritance (e.g., `sanitizer`s still specify allocators off for free usage), but rather to override/arbitrate any conflicting variables among different inherited presets (and the underscore prefix makes it the top of the list alphabetically). We can assume that any potential "arbitrator"s in the future (prefixed with underscore as well) won't overlap with each other so the order between them doesn't matter - as long as they are ahead of any non-underscore-prefixed presets, which is true. I think this way, things are less easy to break. Once it comes to the day that enforcing override ordering is mandatory (let's hope that not!), we can revise these names and introduce number-prefix naming then. How do you think? Plus, I also did some reordering/regrouping/simplification to existing presets the way I see fit. Please also help to check. cc @pitrou as well. Thank you! -- 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]
