Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13956
Change subject: IMPALA-8810: Fix ExplainTest when Orc support is disabled ...................................................................... IMPALA-8810: Fix ExplainTest when Orc support is disabled The problem was that ExplainTest only mocks HdfsPartition.getFileFormat() but not HdfsPartition.getInputFormatDescriptor(), and the latter is called then dereferenced during testScanNodeFsScheme() (only if Orc support is disabled), leading to null pointer exception. I went for the lazy solution of replacing getInputFormatDescriptor().getFileFormat() calls with getFileFormat(). The other solution would have been to mock getInputFormatDescriptor() too, but I think that would have only added unnecessary dependencies to the test. Change-Id: I0d8bcb67e095a63beb9059568ba34b7fd261f233 --- M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M fe/src/main/java/org/apache/impala/planner/SingleNodePlanner.java 3 files changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/56/13956/1 -- To view, visit http://gerrit.cloudera.org:8080/13956 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I0d8bcb67e095a63beb9059568ba34b7fd261f233 Gerrit-Change-Number: 13956 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <[email protected]>
