Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Mon, 25 Apr 2011 17:13:03 +0000 (UTC)
From: "jiraposter@reviews.apache.org (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: 
 <1941121419.115.1303751583239.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <1673211891.74204.1303414986347.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (HBASE-3812) Tidy up naming consistency and
 documentation in coprocessor framework
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

jiraposter@reviews.apache.org commented on HBASE-3812:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/660/
-----------------------------------------------------------

Review request for hbase and Gary Helmling.


Summary
-------

As described at hbase-3812:
* rename BaseRegionObserverCoprocessor -> BaseRegionObserver
* in BaseMasterObserver, rename ObserverContext parameter variable from "env" to "c" or "ctx"
* unnecessary public modifier for methods in RegionObserver interface


This addresses bug hbase-3812.
    https://issues.apache.org/jira/browse/hbase-3812


Diffs
-----

  src/main/java/org/apache/hadoop/hbase/coprocessor/BaseMasterObserver.java 8df6aa4 
  src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserver.java PRE-CREATION 
  src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRegionObserverCoprocessor.java a8c7cb2 
  src/main/java/org/apache/hadoop/hbase/coprocessor/RegionObserver.java 5c5d18d 
  src/test/java/org/apache/hadoop/hbase/coprocessor/SampleRegionWALObserver.java c1fe58d 
  src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java e201b3f 
  src/test/java/org/apache/hadoop/hbase/coprocessor/TestCoprocessorInterface.java 0a07e03 
  src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionObserverStacking.java 20a8d7d 
  src/test/java/org/apache/hadoop/hbase/coprocessor/TestWALCoprocessors.java 27c38f9 
  src/test/java/org/apache/hadoop/hbase/coprocessor/TestWALObserver.java PRE-CREATION 

Diff: https://reviews.apache.org/r/660/diff


Testing
-------

Tests passed locally.


Thanks,

Mingjie


> Tidy up naming consistency and documentation in coprocessor framework
> ---------------------------------------------------------------------
>
>                 Key: HBASE-3812
>                 URL: https://issues.apache.org/jira/browse/HBASE-3812
>             Project: HBase
>          Issue Type: Improvement
>          Components: coprocessors
>            Reporter: Gary Helmling
>            Priority: Critical
>             Fix For: 0.92.0
>
>
> We have a few naming inconsistencies in the coprocessor API and some stale javadocs that have been spotted by Lars George as he digs through it.  We should clean these up before we have an official release and are forced to go through a round of deprecation to make any changes.
> Current items on the list:
>  * rename BaseRegionObserverCoprocessor -> BaseRegionObserver
>  * in BaseMasterObserver, rename ObserverContext parameter variable from "env" to "c" or "ctx"
>  * unnecessary public modifier for methods in RegionObserver interface
> As part of this, we should take a pass through the javadocs and verify they are up to date with what is currently implemented.
> Please tack on other cosmetic changes or inconsistencies as you find them.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira