Re: Review Request: Hive JDBC doesn't support TIMESTAMP column

2012-09-10 Thread Richard Ding
> On Sept. 10, 2012, 9:32 p.m., Carl Steinbach wrote: > > http://svn.apache.org/repos/asf/hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveBaseResultSet.java, > > line 474 > > > > > > Formatting: Missing space

Re: Review Request: Hive JDBC doesn't support TIMESTAMP column

2012-09-10 Thread Richard Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5504/ --- (Updated Sept. 11, 2012, 12:18 a.m.) Review request for hive and Carl Steinbach.

Re: Review Request: Hive JDBC doesn't support TIMESTAMP column

2012-09-10 Thread Carl Steinbach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5504/#review11287 --- http://svn.apache.org/repos/asf/hive/trunk/jdbc/src/java/org/apache/

Re: Review Request: Hive JDBC doesn't support TIMESTAMP column

2012-06-28 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5504/#review8702 --- Looks fine overall. Perhaps we should implement setTimestamp in HiveP

Review Request: Hive JDBC doesn't support TIMESTAMP column

2012-06-21 Thread Richard Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5504/ --- Review request for hive and Carl Steinbach. Description --- See HIVE-2957