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

Posted to commits@hbase.apache.org by br...@apache.org on 2008/02/24 03:02:38 UTC

svn commit: r630566 - /hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java

Author: bryanduxbury
Date: Sat Feb 23 18:02:36 2008
New Revision: 630566

URL: http://svn.apache.org/viewvc?rev=630566&view=rev
Log:
HBASE-458 Shell does not throw an error when you feed it garbage statements.

Modified:
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java?rev=630566&r1=630565&r2=630566&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/Shell.java Sat Feb 23 18:02:36 2008
@@ -115,10 +115,10 @@
 
         long end = System.currentTimeMillis();
         if (rs != null) {
-          if (rs != null && rs.getType() > -1)
+          if (rs != null && rs.getType() > Constants.ERROR_CODE)
             System.out.println(rs.getMsg() +
               executeTime((rs.getType() == 1), start, end));
-          else if (rs.getType() == -9)           
+          else if (rs.getType() == Constants.ERROR_CODE)           
             System.out.println(rs.getMsg());
         }
         queryStr = new StringBuilder();