From 88e19133004e92a30e3eb29096a4d478ecb9fcdd Mon Sep 17 00:00:00 2001 From: Sudheesh Katkam Date: Tue, 18 Oct 2016 14:22:53 -0700 Subject: [PATCH] DRILL-4950: Remove incorrect false condition; consume all empty batches closes #621 --- .../org/apache/drill/jdbc/impl/DrillCursor.java | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java b/exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java index a19f82a2148..08570a8645f 100644 --- a/exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java +++ b/exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java @@ -185,19 +185,6 @@ private boolean nextRowInternally() throws SQLException { qrb.release(); qrb = resultsListener.getNext(); - - // NOTE: It is unclear why this check does not check getRowCount() - // as the loop condition above does. - if ( qrb != null && qrb.getData() == null ) { - // Got another batch with null data--dispose of and report "no more - // rows". - - qrb.release(); - - // NOTE: It is unclear why this returns false but doesn't set - // afterLastRow (as we do when we normally return false). - return false; - } } }