davidyuan1223 commented on PR #6022:
URL: https://github.com/apache/kyuubi/pull/6022#issuecomment-1965965766

   > It is not common to use jdbc to access redis. If the redis jdbc client 
supports the mysql dialect, I think we can use mysql type jdbc engine to access 
redis, like:
   > 
   > ```
   > kyuubi.engine.jdbc.type=MYSQL
   > kyuubi.engine.jdbc.driver.class=RedisDriverClass
   > kyuubi.engine.jdbc.connection.url=jdbc:redis://XXXX
   > ```
   
   The Config Format remains consistent with the current logic. Because 
DriverClass is not mandatory to be itmuch's DriverClass. So curretly jdbc 
config format is
   ```
   kyuubi.engine.type=jdbc
   kyuubi.engine.jdbc.type=redis
   kyuubi.engine.jdbc.driver.class=com.itmuch.redis.jdbc.redis.RedisDriver
   ```
   I have implemented the redis's mysql dialect, in the pr, we need discuss how 
to test redis jdbc driver, because it's data format or metadata is not same as 
common jdbc engine


-- 
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