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

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

Github user harshach commented on the pull request:

    https://github.com/apache/storm/pull/822#issuecomment-163486612
  
    @zhuoliu  @revans2 @d2r Hi,  @arunmahadevan pointed out this jira changed 
log location for worker logs. This is actually not acceptable for our users and 
I don't see any reason for log location to change since we've storm.log.dir and 
its been used in previous releases . If its necessary for this patch to 
function to have the logs in storm-local/worker-artifacts than we should put  
in storm.log.dir and symlink to storm-local/worker-artifacts.  Checking to see 
the reason behind moving the logs there before sending a patch. Thanks.


> Worker Artifacts Directory
> --------------------------
>
>                 Key: STORM-901
>                 URL: https://issues.apache.org/jira/browse/STORM-901
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Zhuo Liu
>             Fix For: 0.11.0
>
>         Attachments: daemon-logs-view.png, worker-logs-view.png
>
>
> At Yahoo we have added in one separate directory that not only stores each 
> topology's every worker's log files, but also provides a place where the 
> worker can place other files which helps with debugging (gc logs and heap 
> dump files).  It is a new directory that soft-links to the current working 
> directory of the worker process.  The oldest files in each worker's artifacts 
> directory are cleaned up if they get too large (exceeding the per-dir quota 
> or the global quota).
> This work is already done, but needs to be put back into open source.



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

Reply via email to