knime-ip / knip

KNIME Image Processing Extension
https://www.knime.com/community/image-processing
50 stars 10 forks source link

Rethink forwarding of SciJava logs #434

Open gab1one opened 7 years ago

gab1one commented 7 years ago

Currently we forward all logs from Scijava and co to the KNIME log via the `KNIPLogService. Because Scijava logs all plugin discoveries asdebug` we end up polluting the KNIME log with messages that are highly irrelevant in most circumstances.

The two solution options I could think of rightaway:

  1. Add settings option to modify the severity of events logged by the KNIPLogService, reduce default to warning and above.
  2. Always ignore messages with a level of debug or lower, as we do not need to SciJava debug info in KNIME.

Additionally we need to think about improvements to the thread safety and general code quality of the KNIPLogService.