You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by bi...@apache.org on 2012/01/09 16:08:13 UTC

svn commit: r1229190 - /incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java

Author: billie
Date: Mon Jan  9 15:08:12 2012
New Revision: 1229190

URL: http://svn.apache.org/viewvc?rev=1229190&view=rev
Log:
ACCUMULO-291 fixed setiter shell command in 1.3

Modified:
    incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java

Modified: incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java
URL: http://svn.apache.org/viewvc/incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java?rev=1229190&r1=1229189&r2=1229190&view=diff
==============================================================================
--- incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java (original)
+++ incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java Mon Jan  9 15:08:12 2012
@@ -1747,12 +1747,12 @@ public class Shell {
       for (IteratorScope scope : scopes) {
         String stem = String.format("%s%s.%s", Property.TABLE_ITERATOR_PREFIX, scope.name(), name);
         log.debug("setting property " + stem + " to " + priority + "," + classname);
-        shellState.connector.tableOperations().setProperty(tableName, stem, priority + "," + classname);
         stem += ".opt.";
         for (Entry<String,String> e : options.entrySet()) {
           log.debug("setting property " + stem + e.getKey() + " to " + e.getValue());
           shellState.connector.tableOperations().setProperty(tableName, stem + e.getKey(), e.getValue());
         }
+        shellState.connector.tableOperations().setProperty(tableName, stem, priority + "," + classname);
       }
     }
     
@@ -2064,7 +2064,7 @@ public class Shell {
         } else {
           if (!Property.isValidZooPropertyKey(property))
             throw new BadArgumentException("Property cannot be modified in zookeeper", fullCommand, fullCommand.indexOf(property));
-
+          
           shellState.connector.instanceOperations().setProperty(property, value);
           log.debug("Successfully set system configuration option");
         }