[ 
https://issues.apache.org/jira/browse/STORM-1673?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15221091#comment-15221091
 ] 

ASF GitHub Bot commented on STORM-1673:
---------------------------------------

GitHub user HeartSaVioR opened a pull request:

    https://github.com/apache/storm/pull/1292

    STORM-1673 log4j2/worker.xml refers old package of LoggerMetricsConsumer

    * fix missing package changes

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/HeartSaVioR/storm STORM-1673-1.x

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1292.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 #1292
    
----
commit dd3d0b2000c41b587cb22ecf16ddfd971c0c1d00
Author: Jungtaek Lim <[email protected]>
Date:   2016-04-01T03:25:48Z

    STORM-1673 log4j2/worker.xml refers old package of LoggerMetricsConsumer
    
    * fix missing package changes

----


> log4j2/worker.xml refers old package of LoggerMetricsConsumer
> -------------------------------------------------------------
>
>                 Key: STORM-1673
>                 URL: https://issues.apache.org/jira/browse/STORM-1673
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-core
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>
> We changed package path from 'backtype.storm' to 'org.apache.storm'. Source 
> codes seem to moved properly, but missed log4j2 configuration, so metric log 
> is logged into worker log file, not metrics file.
> It should be simple patch so I'd like to include this as 1.0.0. If we don't 
> want to include any bugfixes I'm OK to remove epic.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to