Skip to content

Commit

Permalink
Add Commons Logging bridge to broker (#1740)
Browse files Browse the repository at this point in the history
The AWS SDK, which we use for S3 offload uses commons logging as its
logging API. For slf4j to be able to pick up this logs, we need to use
the jcl-over-slf4j bridge. This patch adds the bridge as a dependency
for the broker.

Master Issue: #1511
  • Loading branch information
ivankelly authored and merlimat committed May 7, 2018
1 parent 87440f5 commit cc77e6c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
6 changes: 6 additions & 0 deletions pom.xml
Expand Up @@ -388,6 +388,12 @@ flexible messaging model and an intuitive client API.</description>
<version>${slf4j.version}</version>
</dependency>

<dependency>
<groupId>org.slf4j</groupId>
<artifactId>jcl-over-slf4j</artifactId>
<version>${slf4j.version}</version>
</dependency>

<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-api</artifactId>
Expand Down
5 changes: 5 additions & 0 deletions pulsar-broker/pom.xml
Expand Up @@ -203,6 +203,11 @@
<artifactId>jul-to-slf4j</artifactId>
</dependency>

<dependency>
<groupId>org.slf4j</groupId>
<artifactId>jcl-over-slf4j</artifactId>
</dependency>

<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
Expand Down

0 comments on commit cc77e6c

Please sign in to comment.