Netflix / blitz4j

Logging framework for fast asynchronous logging
Apache License 2.0
561 stars 118 forks source link

Only load properties with the log4j prefix #33

Closed akang31 closed 7 months ago

akang31 commented 7 months ago

As part of the fast property instrumentation effort, we are modifying locations in code that iterate over all properties in order to reduce the amount of noise.

Blitz4j only reads log4j-prefixed properties for overrides, so we change the code here to only read those, instead of first reading then filtering.

This copies up the method from Apache commons configuration https://github.com/apache/commons-configuration/blob/CONFIGURATION_1_10/src/main/java/org/apache/commons/configuration/ConfigurationConverter.java#L105