1.10.2 misses optional imports - breaks AsyncLogger

Description

Version 1.10.2's manifest does not include a lot of optional imports anymore, e.g. com.lmax.disruptor.
This leads to the logger breaking if e.g. Karaf gets configured for async logging log4j2.rootLogger.type = asyncRoot in logging config. Exception below.
Note, that in the export-package, there still is the uses declaration for those missing optional imports.
Adding the missing imports as they were in 1.10.1 fixes the issue and classes can get resolved again.

Note that there are a lot

If you compare 1.10.2 to 1.10.1 those optional imports were removed (does not mean all of them by accident, but at least the ones declared as used are missing):

Environment

None

Activity

Show:
Grzegorz Grzybek
July 18, 2019, 1:32 PM
Edited

Test was fixed by ensuring (Jackson case, but same should work for com.lmax.disruptor) that ifetc/startup.properties contains:

 

If pax-logging-log4j-extra is installed with later start level, pax-logging-log4j2 has to be refreshed to see Jackson packages.

Grzegorz Grzybek
July 18, 2019, 2:06 PM

Also I could use Jackson installed as feature, but after ensuring that pax-logging-log4j2 is refreshed.

Grzegorz Grzybek
July 23, 2019, 8:40 AM

It works without problems in 1.11.x and tests prove it. There was typo in 1.10.x.

Rico Neubauer
July 25, 2019, 1:36 PM

Can confirm that it works with pax-logging-log4j2-extra:1.10.3 - thanks.

Assignee

Jean-Baptiste Onofre

Reporter

Rico Neubauer

Fix versions

Labels

Affects versions

Priority

Major
Configure