----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74131/#review224690 -----------------------------------------------------------
It seems work. But are you sure you want to change the log level of com.mchange from the default warn to error? What's the reason? security-admin/src/main/resources/conf.dist/logback.xml Line 83 (original), 83 (patched) <https://reviews.apache.org/r/74131/#comment313484> If you just want change level and reuse the root logger, just write as that <logger name="com.mchange" level="error"/> It will inherit the appender of root. - Kirby Zhou On 九月 20, 2022, 6:33 p.m., Ramachandran Krishnan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74131/ > ----------------------------------------------------------- > > (Updated 九月 20, 2022, 6:33 p.m.) > > > Review request for ranger, Madhan Neethiraj and Pradeep Agrawal. > > > Bugs: RANGER-3775 > https://issues.apache.org/jira/browse/RANGER-3775 > > > Repository: ranger > > > Description > ------- > > {logger name="com.mchange"} > is used by c3p0 and {logger name="jdbc.connection"} > > is used by log4jdbc > > > <logger name="com.mchange" additivity="false" level="error"> > <appender-ref ref="xa_log_appender"/> > </logger> > > <logger name="jdbc.connection" additivity="false" level="error"> > <appender-ref ref="sql_appender"/> > </logger> > > > Diffs > ----- > > security-admin/src/main/resources/conf.dist/logback.xml 53cdc49cf > > > Diff: https://reviews.apache.org/r/74131/diff/1/ > > > Testing > ------- > > > Thanks, > > Ramachandran Krishnan > >
