GitHub user unsleepy22 opened a pull request:

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

    [Storm-1579] Fix NoSuchFileException when running tests in storm-core

    The old code takes "storm.local.dir" property as logDir while changes in 
STORM-1552 used `ConfigUtils.getLogDir`, which doesn't take this property, 
causing the ultimate event log directory to be "/logs/xxxx" and resulting in 
NoSuchFileException.
    
    Note that the only effective change is in `getLogDir` method, I just 
formatted the whole class.

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

    $ git pull https://github.com/unsleepy22/storm STORM-1579

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

    https://github.com/apache/storm/pull/1155.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 #1155
    
----
commit afd2d525be396c6f430e6a4a13cd1f237496a473
Author: 卫乐 <[email protected]>
Date:   2016-02-24T13:06:25Z

    port backtype.storm.stats to java

commit 52d3b587f07db7dcf66b774531e2face7247c7b6
Author: 卫乐 <[email protected]>
Date:   2016-02-24T13:12:53Z

    add translated java files

commit f61ea0c0196da4f31126f3f96ffb2bf5551a01d2
Author: 卫乐 <[email protected]>
Date:   2016-02-25T02:59:42Z

    move update tuple stat/renderStats methods to corresponding ExecutorStat 
classes

commit 67a5878e5f37ccd317c10ef8dcbd56b9de233997
Author: 卫乐 <[email protected]>
Date:   2016-02-25T05:10:07Z

    Merge https://github.com/apache/storm
    
    Conflicts:
        storm-core/src/clj/org/apache/storm/converter.clj
        storm-core/src/clj/org/apache/storm/daemon/executor.clj
        storm-core/src/clj/org/apache/storm/daemon/nimbus.clj
        storm-core/src/clj/org/apache/storm/stats.clj
        storm-core/src/clj/org/apache/storm/ui/core.clj
        storm-core/test/clj/org/apache/storm/nimbus_test.clj

commit 880134881566427e886b01d44890d22db483f6bd
Author: 卫乐 <[email protected]>
Date:   2016-02-25T05:11:50Z

    merge conflicts from master

commit e5564c0f888e40af2726a645d24cfad0aaeed26a
Author: 卫乐 <[email protected]>
Date:   2016-02-25T07:06:59Z

    added last-error to stats

commit 3fc80c4b0bfc83d2534fab160c72894af044dbc3
Author: 卫乐 <[email protected]>
Date:   2016-02-25T07:25:58Z

    fixed a potential NPE

commit f819b4999eac052840ff69552198b184afa0c1e0
Author: 卫乐 <[email protected]>
Date:   2016-02-26T06:02:45Z

    merge from master

commit e0e9de7d01bb09e6593093cc9324b09f03abb55c
Author: 卫乐 <[email protected]>
Date:   2016-02-26T06:03:35Z

    merge from master

commit db7fe5730953f5d2dc3bb376303d247c1cd1393b
Author: 卫乐 <[email protected]>
Date:   2016-02-26T06:06:48Z

    fix the bug that ClassCastException were thrown when running storm-core 
tests

commit b44233509ef33837a8f4f83882a5b55988336a49
Author: 卫乐 <[email protected]>
Date:   2016-02-26T11:17:23Z

    restore storm-master branch

commit d36cf75b71db5da5fd8f1ad8357075ceafaf33aa
Author: 卫乐 <[email protected]>
Date:   2016-02-26T11:24:29Z

    Merge https://github.com/apache/storm into storm-master

commit 704e5c0179dd12ce29b97998cfd36951fa6bc70e
Author: 卫乐 <[email protected]>
Date:   2016-02-26T11:32:02Z

    add stats.clj

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to