Skip to content
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

Open
archenroot opened this issue Jan 22, 2019 · 5 comments
Open

[feature] - JDBC getImportedKeys method support #4

archenroot opened this issue Jan 22, 2019 · 5 comments

Comments

@archenroot
Copy link

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.

@cdessanti
Copy link

cdessanti commented Jan 22, 2019 via email

@archenroot
Copy link
Author

archenroot commented Jan 22, 2019

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...:
return new ResultSet();

@cdessanti
Copy link

cdessanti commented Jan 22, 2019 via email

@randyzwitch
Copy link

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.

@cdessanti
Copy link

Ok, I'll add the generation of an empty result set with the correct names and datatype on those methods.

cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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.
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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
cdessanti referenced this issue in cdessanti/mapd-core Feb 25, 2019
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
cdessanti referenced this issue in cdessanti/mapd-core Feb 28, 2019
Fixes and enanchements for getTables, getImportedKeys, getExportedKeys. getPrimaryKeys.

Added the helper method getEmptyResultSetWithDesc
dwayneberry referenced this issue in heavyai/heavydb Apr 1, 2019
Fixes and enanchements for getTables, getImportedKeys, getExportedKeys. getPrimaryKeys.

Added the helper method getEmptyResultSetWithDesc
@randyzwitch randyzwitch transferred this issue from heavyai/heavydb Jun 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants