"William A. Rowe Jr." <[email protected]> writes: > Dan Poirier wrote: >> "William A. Rowe Jr." <[email protected]> writes: >> >>> One thing we should refactor is 'debug' logging. Proper debug >>> logging is log early and often, but there is overhead involved >>> in preparing the args and submitting the log request, only to have >>> it fall on deaf ears. >>> >>> If we are doing any significant 2.0 refactoring, toggling the >>> truly-debug log level processing at compile time would be a big win. >> >> Do we have some measurements of how much overhead this adds? > > Of course not, since it varies wildly by what is being logged. But just > the preparation can consume double buffers/invoke a copy with the simple > presence of an LF character.
Looking at log_error_core(), it appears that if the logging level is set to disallow a particular message from being logged, that log_error_core() returns before doing any argument processing. So the overhead would only consist of a few function calls, and my sense would be that saving that wouldn't make a big difference. Of course common sense is notoriously unreliable in making judgments about performance :-), but this does make me even more interested in seeing some evidence of significant improvement before we make major changes in this area at the expense of making problem determination more difficult. Dan
