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

Phabricator commented on HIVE-4324:
-----------------------------------

ashutoshc has requested changes to the revision "HIVE-4324 [jira] ORC Turn off 
dictionary encoding when number of distinct keys is greater than threshold".

  Mostly looks good, except for some minor nits.

INLINE COMMENTS
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/OutStream.java:249 Is it better 
to modify clear to accept compress and suppress arguments ?
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java:768 Good 
to add a javadoc saying this Reader reads strings which doesn't have 
accompanying dictionary.
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java:838 
Similarly here, javadoc of effect : This reader reads dictionary encoded 
strings.
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/StringRedBlackTree.java:166 This 
method could be package private?

REVISION DETAIL
  https://reviews.facebook.net/D12045

BRANCH
  h-4324

ARCANIST PROJECT
  hive

To: JIRA, ashutoshc, omalley

                
> ORC Turn off dictionary encoding when number of distinct keys is greater than 
> threshold
> ---------------------------------------------------------------------------------------
>
>                 Key: HIVE-4324
>                 URL: https://issues.apache.org/jira/browse/HIVE-4324
>             Project: Hive
>          Issue Type: Sub-task
>          Components: File Formats
>    Affects Versions: 0.11.0
>            Reporter: Kevin Wilfong
>            Assignee: Kevin Wilfong
>         Attachments: HIVE-4324.1.patch.txt, HIVE-4324.D12045.1.patch
>
>
> Add a configurable threshold so that if the number of distinct values in a 
> string column is greater than that fraction of non-null values, dictionary 
> encoding is turned off.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to