eskabetxe commented on PR #83: URL: https://github.com/apache/flink-connector-jdbc/pull/83#issuecomment-1857519079
Hi @snuyanzin, I can imagine that the intention would be to have all the db's with a single catalog, but being mandatory the default-database my assumption can be wrong.. Saying that and being one catalog for db, won't be better to add a new option like url (as table have), and deprecate this two. It becomes a little strange to have the db on the two option, and base-url being a full url.. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
