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

chengpan pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/branch-1.7 by this push:
     new 3f153c651 [KYUUBI #3887][FOLLOWUP] Fix kyuubiServerPrincipal logic in 
KyuubiCommands
3f153c651 is described below

commit 3f153c651bbaa2bd76f0428ebc3bab140c218a21
Author: Tianlin Liao <[email protected]>
AuthorDate: Tue Apr 25 02:17:38 2023 +0800

    [KYUUBI #3887][FOLLOWUP] Fix kyuubiServerPrincipal logic in KyuubiCommands
    
    ### _Why are the changes needed?_
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run 
test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    Closes #4764 from lightning-L/kyuubi-3887-followup.
    
    Closes #3887
    
    42a123282 [Tianlin Liao] [KYUUBI #3887][FOLLOWUP] fix kyuubiServerPrincipal 
logic in KyuubiCommands
    
    Authored-by: Tianlin Liao <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
    (cherry picked from commit b0d07f7086556be3d3b4246f7674673d95359f78)
    Signed-off-by: Cheng Pan <[email protected]>
---
 .../src/main/java/org/apache/hive/beeline/KyuubiCommands.java           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiCommands.java 
b/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiCommands.java
index 9ce4cb6f7..9557f2567 100644
--- 
a/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiCommands.java
+++ 
b/kyuubi-hive-beeline/src/main/java/org/apache/hive/beeline/KyuubiCommands.java
@@ -467,7 +467,7 @@ public class KyuubiCommands extends Commands {
 
     beeLine.info("Connecting to " + url);
     if (Utils.parsePropertyFromUrl(url, AUTH_PRINCIPAL) == null
-        || Utils.parsePropertyFromUrl(url, AUTH_KYUUBI_SERVER_PRINCIPAL) == 
null) {
+        && Utils.parsePropertyFromUrl(url, AUTH_KYUUBI_SERVER_PRINCIPAL) == 
null) {
       String urlForPrompt = url.substring(0, url.contains(";") ? 
url.indexOf(';') : url.length());
       if (username == null) {
         username = beeLine.getConsoleReader().readLine("Enter username for " + 
urlForPrompt + ": ");

Reply via email to