You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by kg...@apache.org on 2020/04/22 08:41:10 UTC

[hive] 02/04: HIVE-23249: Prevent infinite loop in TestJdbcWithMiniLlapArrow (Zoltan Haindrich reviewed by Peter Vary)

This is an automated email from the ASF dual-hosted git repository.

kgyrtkirk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git

commit 1186843bbbaa079c22103198f1ac36edfd1fd9a1
Author: Zoltan Haindrich <ki...@rxd.hu>
AuthorDate: Wed Apr 22 08:06:53 2020 +0000

    HIVE-23249: Prevent infinite loop in TestJdbcWithMiniLlapArrow (Zoltan Haindrich reviewed by Peter Vary)
    
    Signed-off-by: Zoltan Haindrich <zh...@cloudera.com>
---
 .../src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlapArrow.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlapArrow.java b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlapArrow.java
index 1aab03d..bc2480a 100644
--- a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlapArrow.java
+++ b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlapArrow.java
@@ -358,7 +358,7 @@ public class TestJdbcWithMiniLlapArrow extends BaseJdbcWithMiniLlap {
 
     // wait for other thread to create the stmt handle
     int count = 0;
-    while (count < 10) {
+    while (++count <= 10) {
       try {
         tKillHolder.throwable = null;
         Thread.sleep(2000);
@@ -380,7 +380,6 @@ public class TestJdbcWithMiniLlapArrow extends BaseJdbcWithMiniLlap {
         stmt2.close();
         break;
       } catch (SQLException e) {
-        count++;
         LOG.warn("Exception when kill query", e);
         tKillHolder.throwable = e;
       }