rgoers commented on pull request #381: URL: https://github.com/apache/logging-log4j2/pull/381#issuecomment-653789389
Now that I look at this I find what was originally implemented rather odd. The setLevel() and setParent() methods throw exceptions and yet the setFilter method is still allowed. It seems to me if setFilter is supported then setLevel should be too (i.e. it would call setLevel on the underlying Log4j Logger). But if setLevel isn't supported then I don't understand why setFilter would be. In any case, I wouldn't have setLevel throw an exception. At most I would just ignore the call. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
