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/04/28 01:54:51 UTC
svn commit: r1331639 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
Author: stack
Date: Fri Apr 27 23:54:51 2012
New Revision: 1331639
URL: http://svn.apache.org/viewvc?rev=1331639&view=rev
Log:
HBASE-5893 Allow spaces in coprocessor conf (aka trim() className)
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java?rev=1331639&r1=1331638&r2=1331639&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java Fri Apr 27 23:54:51 2012
@@ -113,15 +113,15 @@ public abstract class CoprocessorHost<E
protected void loadSystemCoprocessors(Configuration conf, String confKey) {
Class<?> implClass = null;
- // load default coprocessors from configure file
- String defaultCPClasses = conf.get(confKey);
- if (defaultCPClasses == null || defaultCPClasses.length() == 0)
+ // load default coprocessors from configure file
+ String[] defaultCPClasses = conf.getStrings(confKey);
+ if (defaultCPClasses == null || defaultCPClasses.length == 0)
return;
- StringTokenizer st = new StringTokenizer(defaultCPClasses, ",");
+
int priority = Coprocessor.PRIORITY_SYSTEM;
List<E> configured = new ArrayList<E>();
- while (st.hasMoreTokens()) {
- String className = st.nextToken();
+ for (String className : defaultCPClasses) {
+ className = className.trim();
if (findCoprocessor(className) != null) {
continue;
}