HansiChan commented on PR #7720: URL: https://github.com/apache/paimon/pull/7720#issuecomment-4342352799
> Nice! I have a similar change locally that uses SQLAlchemy - but this looks great as it adds fewer dependencies. Is it worth calling it something other than JDBC? It may be confusing to folks who think it uses the JVM underneath for the database connections. Good point. I kept the public catalog type as `jdbc` because it matches Paimon's existing JDBC catalog configuration and lets users reuse the same `metastore=jdbc` / `jdbc:` URI options across engines. To avoid implying that PyPaimon uses JVM JDBC drivers, I updated the implementation and docs to clarify that PyPaimon uses native Python DB-API drivers under the hood. I also renamed the internal connection helper to `_DbApiConnection` and adjusted the driver error messages accordingly. -- 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]
