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

Szehon Ho commented on HIVE-12635:
----------------------------------

Chatted with Aihua on patch, seems simple.

Small comment for consideration, does it make sense to init timestamp variable 
to 0, and then do the loop from 0 to result.rawCells().length?  For cleaner 
code.

And you may know more than me, does this qualify for backward-incompatibility?  
+1 other than those.



> Hive should return the latest hbase cell timestamp as the row timestamp value
> -----------------------------------------------------------------------------
>
>                 Key: HIVE-12635
>                 URL: https://issues.apache.org/jira/browse/HIVE-12635
>             Project: Hive
>          Issue Type: Bug
>          Components: HBase Handler
>    Affects Versions: 2.1.0
>            Reporter: Aihua Xu
>            Assignee: Aihua Xu
>         Attachments: HIVE-12635.patch
>
>
> When hive talks to hbase and maps hbase timestamp field to one hive column,  
> seems hive returns the first cell timestamp instead of the latest one as the 
> timestamp value. 
> Makes sense to return the latest timestamp since adding the latest cell can 
> be  considered an update to the row. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to