davisusanibar commented on issue #34338: URL: https://github.com/apache/arrow/issues/34338#issuecomment-1581240670
> We should just go with the original suggestion: > > > I propose that the default of arrow.memory.debug.allocator be reversed to false by removing the automatic enabling of BaseAllocator.DEBUG on -ea. > > Note that deferring to the logger doesn't help. It's the generation/tracking of the data to be logged that is the issue, not the log messages themselves. I agree with that, the option (2) consider the proposal mention on the issue. My proposal is to migrate also `-Darrow.memory.debug.allocator` to `Logger scope` to our users could handle logger configuration using *.xml file (that offer more granularity) instead of use environment variable. I am starting with remove logic for `-ea` on `BaseAllocator.DEBUG`, then let me know if more changes need to be added. -- 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]
