[ 
https://issues.apache.org/jira/browse/HIVE-1378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12914222#action_12914222
 ] 

HBase Review Board commented on HIVE-1378:
------------------------------------------

Message from: "Steven Wong" <skwsl...@gmail.com>


bq.  On 2010-09-22 12:01:30, Ning Zhang wrote:
bq.  > trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveQueryResultSet.java, 
line 168
bq.  > <http://review.cloudera.org/r/828/diff/4/?file=12387#file12387line168>
bq.  >
bq.  >     constructing an ArrayList for every row is very expensive. Do you 
need a separate copy every row or can share a "cache" among rows?

I will move the ArrayList construction to HiveQueryResultSet's constructor.


bq.  On 2010-09-22 12:01:30, Ning Zhang wrote:
bq.  > 
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazySimpleSerDe.java, 
line 409
bq.  > <http://review.cloudera.org/r/828/diff/4/?file=12399#file12399line409>
bq.  >
bq.  >     Removing this changes the behavior: previously it throws an 
exception and schema mismatch, not it tolerate it. It would be good to remain 
backward compatibility.

Yeah. I don't believe the removal is necessary, and will put it back.


- Steven


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/828/#review1296
-----------------------------------------------------------





> Return value for map, array, and struct needs to return a string 
> -----------------------------------------------------------------
>
>                 Key: HIVE-1378
>                 URL: https://issues.apache.org/jira/browse/HIVE-1378
>             Project: Hadoop Hive
>          Issue Type: Improvement
>          Components: Drivers
>            Reporter: Jerome Boulon
>            Assignee: Steven Wong
>             Fix For: 0.7.0
>
>         Attachments: HIVE-1378.1.patch, HIVE-1378.2.patch, HIVE-1378.3.patch, 
> HIVE-1378.4.patch, HIVE-1378.patch
>
>
> In order to be able to select/display any data from JDBC Hive driver, return 
> value for map, array, and struct needs to return a string

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to