[MDEV-12973] CONNECT Engine table_type=JDBC CATFUNC=TABLES w/Microsoft JDBC driver not working Created: 2017-06-01 Updated: 2017-08-11 Resolved: 2017-06-13 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - Connect |
| Affects Version/s: | 10.1.24 |
| Fix Version/s: | 10.1.26, 10.0.32, 10.2.8 |
| Type: | Bug | Priority: | Major |
| Reporter: | Robert Dyas | Assignee: | Olivier Bertrand |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Environment: |
centOS7 on Google Compute Engine |
||
| Description |
|
When using CONNECT engine with table_type=JDBC and CATFUNC=TABLES and the official Microsoft JDBC driver (latest v6 for jre 8 environment) the list of tables comes back (there are multiple rows) but all the table names are blank. The test case:
And similar for CATFUNC=COLUMNS if I manually specify a valid table on the remote server:
The latest Microsoft JDBC driver can be downloaded here: NOTE: To ease testing, I can provide the password for the above Microsoft server in a private message so you don't have to set up an environment to test. Thank you. |
| Comments |
| Comment by Robert Dyas [ 2017-06-02 ] | ||||||||||||||||||||||||||||||||||||
|
I've created a simple java command line program that you can compile and test - it produces the CORRECT output, so something must be strange with how CONNECT is doing it.
complile: run: produces out like this (CORRECT):
For some reason in CONNECT, the first 3 columns are blank. | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-06-09 ] | ||||||||||||||||||||||||||||||||||||
|
Olivier, Will your fix for table names & column names make it into 10.1.25? | ||||||||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2017-06-10 ] | ||||||||||||||||||||||||||||||||||||
|
I pushed the fix. Whether or not it is included in the distribution of the next versions does not depend on me. | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-06-13 ] | ||||||||||||||||||||||||||||||||||||
|
I was just wondering since the Status for this one is still showing Open rather than Confirmed, In Progress or Fixed (or whatever the right status is). | ||||||||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2017-06-13 ] | ||||||||||||||||||||||||||||||||||||
|
I just forgot to close it! | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-07-05 ] | ||||||||||||||||||||||||||||||||||||
|
I just installed 10.1.25 and this fix is NOT included in that release Was it not included by accident or on purpose? I noticed the change log for 10.1.25 did not list | ||||||||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2017-07-05 ] | ||||||||||||||||||||||||||||||||||||
|
Surely not on purpose. But I had confirmation that CONNECT changes had not been included in Version 10.1.25... by accident? | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-07-12 ] | ||||||||||||||||||||||||||||||||||||
|
Just saw that this is not listed as fixed in the 10.2.7 Changelog. Strange... it looks like other CONNECT changes were included in 10.2.7 but not this one... any idea why not included? | ||||||||||||||||||||||||||||||||||||
| Comment by Sergei Golubchik [ 2017-07-14 ] | ||||||||||||||||||||||||||||||||||||
|
Sorry for this. Because of a merge mistake this fix didn't end up in 10.2.7, it'll only be in 10.2.8. | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-08-11 ] | ||||||||||||||||||||||||||||||||||||
|
Is this now actually included in 10.1.26 ? | ||||||||||||||||||||||||||||||||||||
| Comment by Robert Dyas [ 2017-08-11 ] | ||||||||||||||||||||||||||||||||||||
|
OK, initial testing of NVARCHAR is working, so I guess the changes were integrated just not listed in the change log. Pretty big change not to list! | ||||||||||||||||||||||||||||||||||||
| Comment by Olivier Bertrand [ 2017-08-11 ] | ||||||||||||||||||||||||||||||||||||
|
There has been a git problem in the change log that caused it to be lost. However hopefully all the recent fixes should be included. |