[
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)