You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by mb...@apache.org on 2012/04/14 04:46:01 UTC

svn commit: r1326044 - /hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb

Author: mbautin
Date: Sat Apr 14 02:46:00 2012
New Revision: 1326044

URL: http://svn.apache.org/viewvc?rev=1326044&view=rev
Log:
[HBASE-5335] Fix TestShell error introduced by Dynamic Config

Summary: Error introduced because of an admin.rb conflict when
cherry-picking from apache89 to internal.

Test Plan:
 - mvn test -Dtest=TestShell

Modified:
    hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb

Modified: hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb
URL: http://svn.apache.org/viewvc/hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb?rev=1326044&r1=1326043&r2=1326044&view=diff
==============================================================================
--- hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb (original)
+++ hbase/branches/0.89-fb/src/main/ruby/hbase/admin.rb Sat Apr 14 02:46:00 2012
@@ -454,8 +454,7 @@ module Hbase
       if arg[CONFIG]
         raise(ArgumentError, "#{CONFIG} must be a Hash type") unless arg.kind_of?(Hash)
         for k,v in arg[CONFIG]
-            v = v.to_s unless v.nil?
-          end
+          v = v.to_s unless v.nil?
           family.setValue(k, v)
         end
       end