[ 
https://issues.apache.org/jira/browse/HIVE-26809?focusedWorklogId=838987&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-838987
 ]

ASF GitHub Bot logged work on HIVE-26809:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 13/Jan/23 01:35
            Start Date: 13/Jan/23 01:35
    Worklog Time Spent: 10m 
      Work Description: ayushtkn commented on code in PR #3833:
URL: https://github.com/apache/hive/pull/3833#discussion_r1040356751


##########
llap-server/src/java/org/apache/hadoop/hive/llap/io/decode/OrcEncodedDataConsumer.java:
##########
@@ -61,7 +61,7 @@
 import org.apache.orc.impl.TreeReaderFactory.TreeReader;
 import org.apache.orc.impl.WriterImpl;
 import org.apache.orc.OrcProto;
-
+import org.apache.orc.impl.reader.tree.TypeReader;

Review Comment:
   import order is wrong. Imports should be alphabetically sorted



##########
ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedTreeReaderFactory.java:
##########
@@ -1249,10 +1534,13 @@ public void seek(PositionProvider index) throws 
IOException {
     }
 
     @Override
-    public void nextVector(
-      ColumnVector previousVector, boolean[] isNull, int batchSize) throws 
IOException {
+    public void nextVector(ColumnVector previousVector,
+        boolean[] isNull,
+        final int batchSize,
+        FilterContext filterContext,
+        ReadPhase readPhase) throws IOException {

Review Comment:
   this code doesn't look formatted 



##########
ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedTreeReaderFactory.java:
##########
@@ -224,7 +237,252 @@ private static void skipCompressedIndex(boolean 
isCompressed, PositionProvider i
     index.getNext();
   }
 
-  protected static class StringStreamReader extends StringTreeReader
+  public static class StringDictionaryTreeReaderHive extends TreeReader {

Review Comment:
   Why is this added in scope of upgrade?





Issue Time Tracking
-------------------

    Worklog Id:     (was: 838987)
    Time Spent: 3.5h  (was: 3h 20m)

> Upgrade ORC to 1.8.1
> --------------------
>
>                 Key: HIVE-26809
>                 URL: https://issues.apache.org/jira/browse/HIVE-26809
>             Project: Hive
>          Issue Type: Improvement
>    Affects Versions: 4.0.0
>            Reporter: Dmitriy Fingerman
>            Assignee: Dmitriy Fingerman
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 3.5h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to