> 1. What is the motivation for Driver.java:87 change?
The reason was that there is currently no way to retrieve the result table name in the server. Also, the driver is always returning the result of queries, so, 'result' seems to be a reasonable name for the table. Ideally, we should have a function which converts a schema DDL to a Schema object and we should be able to query the schema object. Right now we just pass the schema string as is to the DynamicSerDe.
> 2. In JdbcSessionState.java what are execString and fileName variables used for?
Right now JdbcSessionState is a dummy class. Its not being used for anything. The plan is to use it later on. I just copied over the class from Cli. I can make it an empty class.
> 3. Shouldn't HiveResultSetMetadata.java be doing something?
The plan was to stage the JDBC implementation. There are a bunch of auto-generated classes which will be used later on.
> 4. In HiveResultSet.java:436 the getDouble function is missing TODO comment? Same is true for getFloat and getInt.
The TODO comment is inside the function body. Isnt that enough?
> 5. For the non implemented functions should we throw a non implemented run time exception instead of just returning 0.
return 0 was auto-generated. I'll change them to throw SQLException instead.