respect beeline verbose

apache · Feb 28, 2024 · fd9c925 · fd9c925
1 parent 95dda6a
commit fd9c925
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiBeeLine.java b/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiBeeLine.java
@@ -191,6 +191,10 @@ protected void processOption(String arg, ListIterator iter) throws ParseExceptio
           };
       cl = beelineParser.parse(options, args);
 
+      if (getOpts().getVerbose()) {
+        ignoreLaunchEngine = false;
+      }
+
       connSuccessful =
           DynMethods.builder("connectUsingArgs")
               .hiddenImpl(BeeLine.class, BeelineParser.class, CommandLine.class)

diff --git a/kyuubi-hive-beeline/src/test/java/org/apache/hive/beeline/KyuubiBeeLineTest.java b/kyuubi-hive-beeline/src/test/java/org/apache/hive/beeline/KyuubiBeeLineTest.java
@@ -19,6 +19,7 @@
 package org.apache.hive.beeline;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
@@ -104,7 +105,13 @@ public void testKyuubiBeelineIgnoreLaunchEngine() {
 
     String[] args3 = {"-u", "badUrl"};
     kyuubiBeeLine.initArgs(args3);
-    assertTrue(!kyuubiBeeLine.isIgnoreLaunchEngine());
+    assertFalse(kyuubiBeeLine.isIgnoreLaunchEngine());
+    kyuubiBeeLine.setIgnoreLaunchEngine(false);
+
+    String[] args4 = {"--ignore-launch-engine", "--verbose", "-f", "test.sql"};
+    kyuubiBeeLine.initArgs(args4);
+    assertFalse(kyuubiBeeLine.isIgnoreLaunchEngine());
+    assert kyuubiBeeLine.getOpts().getScriptFile().equals("test.sql");
     kyuubiBeeLine.setIgnoreLaunchEngine(false);
   }