-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feature] - JDBC getImportedKeys method support #4
Comments
Do you mean DBeaver? It's trivial to add support to such methods
Ottieni Outlook per Android<https://aka.ms/ghei36>
…________________________________
From: ArchenROOT <[email protected]>
Sent: Tuesday, January 22, 2019 1:02:55 PM
To: omnisci/mapd-core
Cc: Subscribed
Subject: [omnisci/mapd-core] [feature] - JDBC getImportedKeys method support (#310)
I am trying to integrate with standard DB IDEs and facing small issue:
Not supported yet, line:1388 class:com.mapd.jdbc.MapDDatabaseMetaData method:getImportedKeys
Not supported yet, line:1388 class:com.mapd.jdbc.MapDDatabaseMetaData method:getImportedKeys
Do you plan to support such JDBC interface method in near future?
Thank you for response.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://github.com/omnisci/mapd-core/issues/310>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AdZQlzVOP4bN4__RUIvMGBjPVEUa79-mks5vFv3vgaJpZM4aMhuR>.
|
Yeah, sure, I will make just empty ResultSet as mapd doesn't work with primary neither foreign keys. But these methods are used by other tools as well.... so just to make them not scream. This issue is just reminder to implement, so I don't forget to replace exception with...: |
Yes tableau as example need an emptyresultset but with the exact column names and datatypes on primary keys.
Have you tried tables with decimal datatypes?
Ottieni Outlook per Android<https://aka.ms/ghei36>
…________________________________
From: ArchenROOT <[email protected]>
Sent: Tuesday, January 22, 2019 1:19:32 PM
To: omnisci/mapd-core
Cc: Candido Dessanti; Comment
Subject: Re: [omnisci/mapd-core] [feature] - JDBC getImportedKeys method support (#310)
Yeah, sure, I will make just empty ResultSet as mapd doesn't work with primary neither foreign keys. But these methods are used by other tools as well.... so just to make them not scream.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/omnisci/mapd-core/issues/310#issuecomment-456379814>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AdZQl_Ha05v1BWKJ24pbPT-9z4Ce2jctks5vFwHUgaJpZM4aMhuR>.
|
Features like these are great areas for community contributions, so if you have the capability to add this functionality, we'd be happy to incorporate it. |
Ok, I'll add the generation of an empty result set with the correct names and datatype on those methods. |
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
Correction of various issues and a little enanchement on getTables method, that, if schemaPattern is specified and there is a match with current connected user, returns only the tables and the views of current user.
This patch correct the issues #163 #164 #310 #311 The new implementation of getPrimaryKey method makes the driver compatible with Tableau 2019.1 Now the GetTables method return a list of table and views of currenct database if the schemaPattern is null or it matches the current database name
This patch correct the issues #163 #164 #310 #311 The new implementation of getPrimaryKey method makes the driver compatible with Tableau 2019.1 Now the GetTables method return a list of table and views of currenct database if the schemaPattern is null or it matches the current database name
Fixes and enanchements for getTables, getImportedKeys, getExportedKeys. getPrimaryKeys. Added the helper method getEmptyResultSetWithDesc
Fixes and enanchements for getTables, getImportedKeys, getExportedKeys. getPrimaryKeys. Added the helper method getEmptyResultSetWithDesc
I am trying to integrate with standard DB IDEs and facing small issue:
Do you plan to support such JDBC interface method in near future?
Thank you for response.
The text was updated successfully, but these errors were encountered: