nathanmarz/storm — Issue #144: Standardize on SLF4J as a logging implementation
Author: kyrill007
Created: 2012-02-28 01:59:41+00:00
State: open

It appears that storm/lib directory includes both commons-logging-1.1.1.jar and slf4j-api-1.5.8.jar (with log4j being an underlying logging implementation). Those 2 don't really play well together in all scenarios. Basically both will end up instantiating their own log4j instances and log separately to the same set of files.

It is better to standardize on SLF4J and to satisfy commons-logging runtime dependency it is better to include ' jcl-over-slf4j' jar from the SLF4J project that is effectively a bridge implementation of all commons-logging interface APIs. This way all existing projects that are dependent on commons-logging will still work by delegating directly to SLF4J.

Upgrading to SLF4J 1.6.4 may also be considered.