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

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

GitHub user unsleepy22 opened a pull request:

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

    [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 because permission is denied to create /logs directory.
    
    The change of code causes 3 test cases to fail in supervisor_test, so I 
changed them accordingly to make all tests pass.

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/1161.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 #1161
    
----
commit 034f0cf107403100650d6eb65e7168f62133864a
Author: 卫乐 <[email protected]>
Date:   2016-02-28T14:33:21Z

    fix STORM-1579, checks storm.local.dir property/conf when getting storm log 
dir

commit 504c11b8ead80e186ff0de83dbdece2337cd1162
Author: 卫乐 <[email protected]>
Date:   2016-02-28T14:42:56Z

    append "/logs" to "storm.local.dir" property when non-null

----


> Fix topology event sampling log directory 
> ------------------------------------------
>
>                 Key: STORM-1552
>                 URL: https://issues.apache.org/jira/browse/STORM-1552
>             Project: Apache Storm
>          Issue Type: Bug
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Arun Mahadevan
>            Assignee: Arun Mahadevan
>             Fix For: 1.0.0
>
>
> Run a topology and enable event inspection by clicking "Debug" from UI. The 
> events are logged under 
> "storm-local/workers-artifacts/{storm-id}/port/events.log". In the spout/bolt 
> details page, the "events" link does not display the log file.
> The events.log should be kept under 
> logs/workers-artifacts/{storm-id}/{port}/events.log so that its viewable via 
> logviewer.



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

Reply via email to