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

Subru Krishnan commented on YARN-5525:
--------------------------------------

Thanks [~botong] for your response. 

bq. One reason that we also want to customize LogAggregationService is to allow 
a different root log directory to upload logs to per app

I want to make sure I grok your scenario fully (and also ensure we are not over 
engineering :)) - do you have other concrete reasons than the one above for the 
proposal? The reason I am asking is I feel it'll be more easier to extend the 
existing {{LogAggregationService}} to allow specifying per-app root dirs than 
overriding the entire service.

> Make log aggregation service class configurable
> -----------------------------------------------
>
>                 Key: YARN-5525
>                 URL: https://issues.apache.org/jira/browse/YARN-5525
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: log-aggregation
>            Reporter: Giovanni Matteo Fumarola
>            Assignee: Botong Huang
>            Priority: Minor
>         Attachments: YARN-5525.v1.patch, YARN-5525.v2.patch, 
> YARN-5525.v3.patch
>
>
> Make the log aggregation class configurable and extensible, so that 
> alternative log aggregation behaviors like app specific log aggregation 
> directory, log aggregation format can be implemented and plugged in.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to