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

Sreepathi Prasanna commented on CHUKWA-715:
-------------------------------------------

       TestRestAdaptor.java and OozieAdaptor.java are missing license file.

Have added the license files. 

       ChukwaConfiguration class is designed to read ChukwaConfiguration files. 
It looks like ChukwaUtil does something similar but not exactly. I think it 
could be optimized to  use      ChukwaConfiguration class to reduce some 
duplications.

the readConfig() method in ChukwaAgent.java might be reused in many places. To 
avoid code duplication, i have placed this method in ChukwaUtil class. Now both 
oozieAdaptor and chukwaAgent classes can read the chukwaconfiguration code 
using this readConfiguration method in chukwaUtil class. ( essentially, 
refactoring)

> Implement Oozie Adaptor to collect oozie instrumentation info
> -------------------------------------------------------------
>
>                 Key: CHUKWA-715
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-715
>             Project: Chukwa
>          Issue Type: New Feature
>          Components: Data Collection
>    Affects Versions: 0.6.0
>            Reporter: Sreepathi Prasanna
>            Assignee: Sreepathi Prasanna
>             Fix For: 0.6.0
>
>         Attachments: chukwa-715.patch.0, chukwa-715.patch.1
>
>
> Oozie exposes metrics via rest urls. And hence this feature will allow to 
> collect the instrumentation information to chukwa and monitor oozie. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to