Trivial patch that does just that.

The getLeases has to be added to RegionServerServices? Otherwise +1 on patch.

@Stack: Yep, because that is all a RegionObserver can see (through the RegionCoprocessorEnvironment). I think it also makes sense, because it is a service provided by the RegionServer.

@Lars OK boss

V2 also fixes up the mock implementations of RSServices and add some simple test code
(which just makes sure that getLeases() and getLock(...) is accessible and can be called from a RegionObserver implementation).

BTW. this also came up in an internal Salesforce discussion and we need something the leases for some internal use cases.

Going to commit soon if nobody objects to the new API(s) exposed to CPs.

+1.

Committed to 0.94 and 0.96.

One part I do notice now 0.94 that Leases.createLease does not take a experation time (but it does in 0.96).
So I would like to add an addendum here and add that to 0.94 as well.

I.e. change

 public void createLease(String leaseName, final LeaseListener listener) 

to the 0.96 version:

 public void createLease(String leaseName, int leaseTimeoutPeriod, final LeaseListener listener) 

At the same time we can tackle this in another jira when it turns out to actually be a problem. As it stands in 0.94 all leases take default leasePeriod configured for this region server.

Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #127 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/127/)
HBASE-6522 Expose locks and leases to Coprocessors (Revision 1371024)

Result = FAILURE
larsh :
Files :

• /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
• /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
• /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerServices.java
• /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
• /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/master/MockRegionServer.java
• /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/MockRegionServerServices.java

Integrated in HBase-0.94-security #48 (See https://builds.apache.org/job/HBase-0.94-security/48/)
HBASE-6522 Expose locks and leases to Coprocessors (Revision 1371026)

Result = FAILURE
larsh :
Files :

• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerServices.java
• /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
• /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/util/MockRegionServerServices.java

Integrated in HBase-0.94-security-on-Hadoop-23 #7 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/7/)
HBASE-6522 Expose locks and leases to Coprocessors (Revision 1371026)

Result = FAILURE
larsh :
Files :

• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
• /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerServices.java
• /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/coprocessor/SimpleRegionObserver.java
• /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/util/MockRegionServerServices.java

Fix up after bulk move overwrote some 0.94.2 fix versions w/ 0.95.0 (Noticed by Lars Hofhansl)