hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Shelukhin (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-11259) LLAP: clean up ORC dependencies part 1
Date Tue, 21 Jul 2015 22:34:04 GMT

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

Sergey Shelukhin commented on HIVE-11259:
-----------------------------------------

1) These classes existed before, I will see if they can be merged. 
2) I don't think removing intrinsic linked list is a good idea, there are problems with java
LinkedList - it's hard to keep pointer to an element, as soon as list is modified all the
iterators are invalidated. So for example reading multiple columns RG by RG, keeping the pointer
to end of last RG (where nexr RG, that may be in separate buffer due to SARG filtering, starts)
becomes impossible - as soon as one column read replaces buffer chunk with cache chunk, pointers
for all other columns become invalid. TreeMap will have the same problem as far as I can tell.
It's really not that complicated to have a linked list, if Java was a real programming language
we could even make it an aspect sort of thing via multiple inheritance :)
3) Ok

> LLAP: clean up ORC dependencies part 1
> --------------------------------------
>
>                 Key: HIVE-11259
>                 URL: https://issues.apache.org/jira/browse/HIVE-11259
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HIVE-11259.patch
>
>
> Before there's storage handler module, we can clean some things up



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

Mime
View raw message