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 43e3c0cf2 [KYUUBI #4954] [DOCS] Fix variable reference in 
quick_start_with_jdbc
43e3c0cf2 is described below

commit 43e3c0cf249fafdd6e6dad0202983967dd235d92
Author: 鱼肚 <[email protected]>
AuthorDate: Tue Jun 13 16:16:23 2023 +0800

    [KYUUBI #4954] [DOCS] Fix variable reference in quick_start_with_jdbc
    
    fix typo, replace kyuubiJdbcUrl with kyuubiJdbcUrlTemplate.
    
    ### _Why are the changes needed?_
    fix typo, should be String.format(kyuubiJdbcUrlTemplate, ...)
    
    ### _How was this patch tested?_
    just fix typo, no test need.
    - [ ] 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 #4954 from uicosp/patch-1.
    
    Closes #4954
    
    8abf04d87 [鱼肚] Update quick_start_with_jdbc.md
    
    Authored-by: 鱼肚 <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
    (cherry picked from commit 32bb71efcc5aa96a6c5998aaf6dfc97b390276d4)
    Signed-off-by: Cheng Pan <[email protected]>
---
 docs/quick_start/quick_start_with_jdbc.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/quick_start/quick_start_with_jdbc.md 
b/docs/quick_start/quick_start_with_jdbc.md
index c7001db64..862e22fa1 100644
--- a/docs/quick_start/quick_start_with_jdbc.md
+++ b/docs/quick_start/quick_start_with_jdbc.md
@@ -80,7 +80,7 @@ public class KyuubiJDBCDemo {
     String clientPrincipal = args[0]; // Kerberos principal
     String clientKeytab = args[1];    // Keytab file location
     String serverPrincipal = arg[2];  // Kerberos principal used by Kyuubi 
Server
-    String kyuubiJdbcUrl = String.format(kyuubiJdbcUrl, clientPrincipal, 
clientKeytab, serverPrincipal);
+    String kyuubiJdbcUrl = String.format(kyuubiJdbcUrlTemplate, 
clientPrincipal, clientKeytab, serverPrincipal);
     try (Connection conn = DriverManager.getConnection(kyuubiJdbcUrl)) {
       try (Statement stmt = conn.createStatement()) {
         try (ResultSet rs = st.executeQuery("show databases")) {

Reply via email to