Double error logging with root handler set on error level

[aiguo-ebay]

In order to get rid of noisy logs from 3rd party libs, we set root handler to the error level then set it to info level for the application package
We upgrades to latest version [3.16.1], still had the same issue.

  override def load(): Unit = {
    OutputFormat.default = ASCIIOutputFormat
    Formatter.default = formatterWithoutThread
    // Set the root handler to the error level
    Logger.root.clearHandlers().withHandler(minimumLevel = Some(Level.Error)).replace()

    // Set the com.xxx packages to the info level
    Logger("com.xxx").withHandler(minimumLevel = Some(Level.Info)).replace()
}

example:

2025.04.08 15:00:54:298 ERROR c.e.f.i.b.g.AlphaIndexBuilderAbstract.buildAlphaIndex:175 - Error while processing rowKey [13340046280626274300], pkId [170032530777]
2025.04.08 15:00:54:298 ERROR c.e.f.i.b.g.AlphaIndexBuilderAbstract.buildAlphaIndex:175 - Error while processing rowKey [13340046280626274300], pkId [170032530777]

[aiguo-ebay]

@darkfrog26 please let me know if you need help to reproduce the issue.

[darkfrog26]

That's working exactly as designed. Realize that the reason you're getting two entries logged is because the root logger handler you create is logging because it's an error and "com.xxx" is is getting logged because error is higher priority than info.

There are a few ways of changing this if you only want one logging entry, but the easiest is just to add .orphan() before .replace() on Logger("com.xxx")

[aiguo-ebay]

will it remove the duped error log?

Logger("com.xxx").withHandler(minimumLevel = Some(Level.Info)).orphan().replace()

[darkfrog26]

Yes, it should

[aiguo-ebay]

thanks @darkfrog26 , it works.