You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by jnioche <gi...@git.apache.org> on 2017/01/26 12:25:26 UTC
[GitHub] storm pull request #1896: STORM-2326 Upgrade log4j and slf4j
GitHub user jnioche opened a pull request:
https://github.com/apache/storm/pull/1896
STORM-2326 Upgrade log4j and slf4j
Ran ` mvn clean install -DskipTests` without problems
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jnioche/storm STORM-2326
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1896.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1896
----
commit 5f9bffbbb15f9964612b48e91db9b24c710a251b
Author: Julien Nioche <ju...@digitalpebble.com>
Date: 2017-01-26T12:23:35Z
STORM-2326 Upgrade log4j and slf4j
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm issue #1896: STORM-2326 Upgrade log4j and slf4j
Posted by jnioche <gi...@git.apache.org>.
Github user jnioche commented on the issue:
https://github.com/apache/storm/pull/1896
See issue with #1900 - closing for now
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm issue #1896: STORM-2326 Upgrade log4j and slf4j
Posted by jnioche <gi...@git.apache.org>.
Github user jnioche commented on the issue:
https://github.com/apache/storm/pull/1896
As far as I understand, Storm uses SLF4J + LOG4J. I could not find a way to run a topology where one of the dependencies relies on a more recent version of log4j (Elasticsearch 5). A similar problem was reported on https://issues.apache.org/jira/browse/STORM-1386.
Upgrading the dependencies in Storm solved the problem.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request #1896: STORM-2326 Upgrade log4j and slf4j
Posted by jnioche <gi...@git.apache.org>.
Github user jnioche closed the pull request at:
https://github.com/apache/storm/pull/1896
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---