David Mollitor created ORC-840:
----------------------------------
Summary: Remove Superfluous Array Fill in RecordReaderImpl
Key: ORC-840
URL: https://issues.apache.org/jira/browse/ORC-840
Project: ORC
Issue Type: Improvement
Reporter: David Mollitor
Assignee: David Mollitor
{code:java}
public static int[] mapSargColumnsToOrcInternalColIdx(
List<PredicateLeaf> sargLeaves,
SchemaEvolution evolution) {
int[] result = new int[sargLeaves.size()];
Arrays.fill(result, -1);
for(int i=0; i < result.length; ++i) {
String colName = sargLeaves.get(i).getColumnName();
result[i] = findColumns(evolution, colName);
}
return result;
}
{code}
There is no reason here to pre-populate {{result}} with -1 values. They are
promptly overwritten in the following loop.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
