You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Marios Trivyzas (Jira)" <ji...@apache.org> on 2023/04/03 08:06:00 UTC
[jira] [Commented] (FLINK-31553) Choose Catalog, non-url based
[ https://issues.apache.org/jira/browse/FLINK-31553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17707805#comment-17707805 ]
Marios Trivyzas commented on FLINK-31553:
-----------------------------------------
Imho, I don't think that this requires a FLIP, it's a simple change/addition.
I created a discussion thread in the list, instead.
> Choose Catalog, non-url based
> -----------------------------
>
> Key: FLINK-31553
> URL: https://issues.apache.org/jira/browse/FLINK-31553
> Project: Flink
> Issue Type: Improvement
> Components: Connectors / JDBC
> Reporter: Marios Trivyzas
> Priority: Major
>
> Currently. the {*}Catalog{*}{*}/Dialect{*} (etc.) is chosen automatically based on the URL provided. This takes place in {*}{color:#000000}JdbcDialectFactory{color}#load{*}, using the: {*}{color:#000000}JdbcDialectFactory{color}#acceptsURL{*}, so for a URL *jdbc:postgresql://...* the PostgresCatalog/Dialect is used. *CrateDB* used the same driver and URL but needs it's own Catalog/Dialect (etc.) as it has it's own stack of internal tables, type sytem, etc. (see https://issues.apache.org/jira/browse/FLINK-31551) So if a user wants to use *CrateDB,* currently, needs to use the legacy CrateDB driver which uses the jdbc:crate://... url.
> Ideally, there should be another way, not only url based, that allows the user to use a given url, but choose the dialect manually. It could be some parameter on the catalog definition or even a special URL parameter like *?dialect=CrateDB* or so.
>
> Other DBs that use implement the PostgreSQL wire protocol and are compatible with the PostgreSQL JDBC driver, but need their own Catalog/Dialect etc. would benefit from this. (e.g.: CockroachDB)
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)