hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Elliott Clark (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HBASE-6412) Move external servers to metrics2 (thrift,thrift2,rest)
Date Tue, 04 Sep 2012 21:12:08 GMT

     [ https://issues.apache.org/jira/browse/HBASE-6412?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Elliott Clark updated HBASE-6412:
---------------------------------

    Attachment: HBASE-6412-1.patch

Talked to Stack and he and I agree that code cleanliness is the most important, so requiring
mvn package as the replacement for mvn compile seems like a fair trade.  Because of that I
have updated the developer docbook.

Latest patch adds an init method so that tests have the ability to re-set sources.  This allows
all of the sources to be singletons and still be pretty testable. (hadoop2 really complains
if sources aren't singletons.)

Added a lot of comments.

Fixed file headers.

Moved constants to the interfaces so all strings are the same.

Changed test-patch so subsequent patches will use package rather than compile. (Note this
won't apply for this patch as hadoop qa starts the script before patching.)
                
> Move external servers to metrics2 (thrift,thrift2,rest)
> -------------------------------------------------------
>
>                 Key: HBASE-6412
>                 URL: https://issues.apache.org/jira/browse/HBASE-6412
>             Project: HBase
>          Issue Type: Sub-task
>    Affects Versions: 0.96.0
>            Reporter: Elliott Clark
>            Assignee: Elliott Clark
>            Priority: Blocker
>         Attachments: HBASE-6412-0.patch, HBASE-6412-1.patch
>
>
> Implement metrics2 for all the external servers:
> * Thrift
> * Thrift2
> * Rest

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message