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 2009/08/18 20:59:50 UTC

svn commit: r805537 - /hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java

Author: stack
Date: Tue Aug 18 18:59:50 2009
New Revision: 805537

URL: http://svn.apache.org/viewvc?rev=805537&view=rev
Log:
HBASE-1768 REST server has upper limit of 5k PUT; second fix

Modified:
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java?rev=805537&r1=805536&r2=805537&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/rest/Dispatcher.java Tue Aug 18 18:59:50 2009
@@ -374,14 +374,13 @@
           return new byte[0];
         }
       }
-      char[] c;// 40 characters * sizeof(UTF16)
-      while (r.ready()) {
+      char [] c;// 40 characters * sizeof(UTF16)
+      while (true) {
     	c = new char[bufferLength]; 
         int n = r.read(c, 0, bufferLength);
-        resultant += new String(c);
-        if (n != bufferLength) {
-          break;
-        } else if (resultant.length() > maxLength) {
+        if (n == -1) break;
+        resultant += new String(c, 0, n);
+        if (resultant.length() > maxLength) {
           resultant = resultant.substring(0, maxLength);
           break;
         }