Skip to content

Commit

Permalink
Remove unused systemMemoryContext from OrcBatchRecordReader
Browse files Browse the repository at this point in the history
  • Loading branch information
Ying Su authored and mbasmanova committed Aug 28, 2019
1 parent bd304e8 commit 47423bb
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public OrcBatchRecordReader(
// doesn't have a local buffer. All non-leaf level StreamReaders' (e.g. MapStreamReader, LongStreamReader,
// ListStreamReader and StructStreamReader) instance sizes were not counted, because calling setBytes() in
// their constructors is confusing.
createStreamReaders(orcDataSource, types, hiveStorageTimeZone, includedColumns, systemMemoryUsage.newAggregatedMemoryContext()),
createStreamReaders(orcDataSource, types, hiveStorageTimeZone, includedColumns),
predicate,
numberOfRows,
fileStripes,
Expand Down Expand Up @@ -153,8 +153,7 @@ private static BatchStreamReader[] createStreamReaders(
OrcDataSource orcDataSource,
List<OrcType> types,
DateTimeZone hiveStorageTimeZone,
Map<Integer, Type> includedColumns,
AggregatedMemoryContext systemMemoryContext)
Map<Integer, Type> includedColumns)
{
List<StreamDescriptor> streamDescriptors = createStreamDescriptor("", "", 0, types, orcDataSource).getNestedStreams();

Expand Down

0 comments on commit 47423bb

Please sign in to comment.