You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2012/01/07 23:20:45 UTC

svn commit: r1228742 - /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java

Author: stack
Date: Sat Jan  7 22:20:45 2012
New Revision: 1228742

URL: http://svn.apache.org/viewvc?rev=1228742&view=rev
Log:
HBASE-5052 The path where a dynamically loaded coprocessor jar is copied on the local file system depends on the region name (and implicitly, the start key)

Modified:
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java?rev=1228742&r1=1228741&r2=1228742&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java Sat Jan  7 22:20:45 2012
@@ -111,7 +111,7 @@ public class RegionCoprocessorHost
       final RegionServerServices rsServices, final Configuration conf) {
     this.rsServices = rsServices;
     this.region = region;
-    this.pathPrefix = this.region.getRegionNameAsString().replace(',', '_');
+    this.pathPrefix = Integer.toString(this.region.getRegionInfo().hashCode());
 
     // load system default cp's from configuration.
     loadSystemCoprocessors(conf, REGION_COPROCESSOR_CONF_KEY);