silencily opened a new pull request, #6037:
URL: https://github.com/apache/kyuubi/pull/6037

   # :mag: Description
   ## Issue References 🔗
   <!-- Append the issue number after #. If there is no issue for you to link 
create one or -->
   <!-- If there are no issues to link, please provide details here. -->
   
   This pull request fixes create Jdbc Engine error:
   
    java.lang.IllegalArgumentException: Could not find a JDBC connection 
provider with name 
'org.apache.kyuubi.engine.jdbc.mysql.MySQLConnectionProvider' that can handle 
the specified driver and options. Available providers are 
[org.apache.kyuubi.engine.jdbc.doris.DorisConnectionProvider@57459491, 
org.apache.kyuubi.engine.jdbc.mysql.MySQLConnectionProvider@dcfda20, 
org.apache.kyuubi.engine.jdbc.phoenix.PhoenixConnectionProvider@5c87bfe2, 
org.apache.kyuubi.engine.jdbc.postgresql.PostgreSQLConnectionProvider@51cd7ffc, 
org.apache.kyuubi.engine.jdbc.starrocks.StarRocksConnectionProvider@40f1be1b]
   
   ## Describe Your Solution 🔧
   
   `  val ENGINE_JDBC_CONNECTION_PROVIDER: OptionalConfigEntry[String] =
       buildConf("kyuubi.engine.jdbc.connection.provider")
         .doc("A JDBC connection provider plugin for the Kyuubi Server " +
           "to establish a connection to the JDBC URL." +
           " The configuration value should be a subclass of " +
           "`org.apache.kyuubi.engine.jdbc.connection.JdbcConnectionProvider`. 
" +
           "Kyuubi provides the following built-in implementations: " +
           "<li>doris: For establishing Doris connections.</li> " +
           "<li>mysql: For establishing MySQL connections.</li> " +
           "<li>phoenix: For establishing Phoenix connections.</li> " +
           "<li>postgresql: For establishing PostgreSQL connections.</li>" +
           "<li>starrocks: For establishing StarRocks connections.</li>")
         .version("1.6.0")
         .stringConf
         .transform {
           case "Doris" | "doris" | "DorisConnectionProvider" =>
             "org.apache.kyuubi.engine.jdbc.doris.DorisConnectionProvider"
           case "MySQL" | "mysql" | "MySQLConnectionProvider" =>
             "org.apache.kyuubi.engine.jdbc.mysql.MySQLConnectionProvider"
           case "Phoenix" | "phoenix" | "PhoenixConnectionProvider" =>
             "org.apache.kyuubi.engine.jdbc.phoenix.PhoenixConnectionProvider"
           case "PostgreSQL" | "postgresql" | "PostgreSQLConnectionProvider" =>
             
"org.apache.kyuubi.engine.jdbc.postgresql.PostgreSQLConnectionProvider"
           case "StarRocks" | "starrocks" | "StarRocksConnectionProvider" =>
             
"org.apache.kyuubi.engine.jdbc.starrocks.StarRocksConnectionProvider"
           case other => other
         }
         .createOptional`
   
   because the provider name transform fully qualified class name,but  class 
simple name is compared
   
   
   ## Types of changes :bookmark:
   <!--- What types of changes does your code introduce? Put an `x` in all the 
boxes that apply: -->
   - [x] Bugfix (non-breaking change which fixes an issue)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   
   ## Test Plan 🧪
   
   #### Behavior Without This Pull Request :coffin:
   
   
   #### Behavior With This Pull Request :tada:
   
   I have passed this pull request at local integration testing
   
   #### Related Unit Tests
   
   
   ---
   
   # Checklist 📝
   <!--- Go over all the following points, and put an `x` in all the boxes that 
apply. -->
   <!--- If you're unsure about any of these, don't hesitate to ask. We're here 
to help! -->
   
   - [ ] This patch was not authored or co-authored using [Generative 
Tooling](https://www.apache.org/legal/generative-tooling.html)
   
   **Be nice. Be informative.**
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to