[ https://issues.apache.org/jira/browse/HIVE-1126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891312#action_12891312 ]
John Sichi commented on HIVE-1126: ---------------------------------- Regarding the synchronization: Squirrel really should not be calling onto multiple objects from the same JDBC connections from different threads (except in the special case of cancel). In general, JDBC drivers don't make a guarantee of thread safety within the same connection. If it is expecting this guarantee, then in general the synchronization would need to be added at a higher level (on the JDBC connection object itself, regardless of which object is receiving the API call) since over time we could be adding more client-side state. Creating a new client connection is not a good approach since we want the session state shared within a connection. I suggest we get the current work committed without any synchronization, and then open up a separate JIRA to investigate further. Regarding Carl's review comments: all of them look correct to me, so could you take care of them as part of the next rev of the patch? > Missing some Jdbc functionality like getTables getColumns and > HiveResultSet.get* methods based on column name. > -------------------------------------------------------------------------------------------------------------- > > Key: HIVE-1126 > URL: https://issues.apache.org/jira/browse/HIVE-1126 > Project: Hadoop Hive > Issue Type: Improvement > Components: Clients > Affects Versions: 0.7.0 > Reporter: Bennie Schut > Assignee: Bennie Schut > Priority: Minor > Fix For: 0.7.0 > > Attachments: HIVE-1126-1.patch, HIVE-1126-2.patch, HIVE-1126.patch, > HIVE-1126_patch(0.5.0_source).patch > > > I've been using the hive jdbc driver more and more and was missing some > functionality which I added > HiveDatabaseMetaData.getTables > Using "show tables" to get the info from hive. > HiveDatabaseMetaData.getColumns > Using "describe tablename" to get the columns. > This makes using something like SQuirreL a lot nicer since you have the list > of tables and just click on the content tab to see what's in the table. > I also implemented > HiveResultSet.getObject(String columnName) so you call most get* methods > based on the column name. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.