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

Posted to commits@sqoop.apache.org by bl...@apache.org on 2012/02/28 00:42:46 UTC

svn commit: r1294396 - /incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java

Author: blee
Date: Mon Feb 27 23:42:45 2012
New Revision: 1294396

URL: http://svn.apache.org/viewvc?rev=1294396&view=rev
Log:
SQOOP-425 Sqoop invokes ResultSet.isLast() which causes Sybase JDBC driver to hang

Modified:
    incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java

Modified: incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java
URL: http://svn.apache.org/viewvc/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java?rev=1294396&r1=1294395&r2=1294396&view=diff
==============================================================================
--- incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java (original)
+++ incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java Mon Feb 27 23:42:45 2012
@@ -272,8 +272,7 @@ public class DBRecordReader<T extends DB
    */
   protected boolean isDone() {
     try {
-      return this.results != null
-          && (results.isLast() || results.isAfterLast());
+      return this.results != null && results.isAfterLast();
     } catch (SQLException sqlE) {
       return true;
     }