dongjoon-hyun commented on a change in pull request #635:
URL: https://github.com/apache/orc/pull/635#discussion_r566566425
##########
File path: java/core/src/java/org/apache/orc/impl/TreeReaderFactory.java
##########
@@ -663,41 +691,46 @@ public void checkEncoding(OrcProto.ColumnEncoding
encoding) throws IOException {
}
@Override
- public void startStripe(StripePlanner planner) throws IOException {
- super.startStripe(planner);
+ public void startStripe(StripePlanner planner, ReadLevel readLevel) throws
IOException {
+
+ super.startStripe(planner, readLevel);
StreamName name = new StreamName(columnId,
OrcProto.Stream.Kind.DATA);
reader = createIntegerReader(planner.getEncoding(columnId).getKind(),
planner.getStream(name), true, context);
}
@Override
- public void seek(PositionProvider[] index) throws IOException {
- seek(index[columnId]);
+ public void seek(PositionProvider[] index, ReadLevel readLevel) throws
IOException {
+ seek(index[columnId], readLevel);
}
@Override
- public void seek(PositionProvider index) throws IOException {
- super.seek(index);
+ public void seek(PositionProvider index, ReadLevel readLevel) throws
IOException {
+
+ super.seek(index, readLevel);
reader.seek(index);
}
@Override
public void nextVector(ColumnVector previousVector,
boolean[] isNull,
final int batchSize,
- FilterContext filterContext) throws IOException {
+ FilterContext filterContext,
+ ReadLevel readLevel) throws IOException {
+
Review comment:
Please remove all empty line insertion.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
