You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Patrick Hunt <ph...@apache.org> on 2011/07/30 00:43:00 UTC

Review Request: HIVE-1631: JDBC driver returns wrong precision, scale, or column size for some data types

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1226/
-----------------------------------------------------------

Review request for hive and Carl Steinbach.


Summary
-------

This patch addresses the specified methods based on my reading of the spec.

It's not at all clear to me that I've interpreted the spec correctly, so please do chime in if you think something is wrong - for example: in my update Hive string columns have a size of Integer.MAX_VALUE given Hive has no bounds on length. Also take a look at precision and size for float/double.

This patch also fixes a problem with the metadata returned by DatabaseMetaData.getColumns(...).getMetaData()


This addresses bug HIVE-1631.
    https://issues.apache.org/jira/browse/HIVE-1631


Diffs
-----

  jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveDatabaseMetaData.java 0e106be 
  jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveResultSetMetaData.java cf651b6 
  jdbc/src/java/org/apache/hadoop/hive/jdbc/JdbcColumn.java 9182744 
  jdbc/src/java/org/apache/hadoop/hive/jdbc/Utils.java PRE-CREATION 
  jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java 91af351 

Diff: https://reviews.apache.org/r/1226/diff


Testing
-------

I updated the tests and also viewed the result of these changes via Squirrel


Thanks,

Patrick