[ 
https://issues.apache.org/jira/browse/IMPALA-8810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16897055#comment-16897055
 ] 

ASF subversion and git services commented on IMPALA-8810:
---------------------------------------------------------

Commit ffb78e24d91fadd3ffb79692211a6208a9eaedf1 in impala's branch 
refs/heads/master from Csaba Ringhofer
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=ffb78e2 ]

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
Reviewed-on: http://gerrit.cloudera.org:8080/13956
Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>


> ExplainTest.testScanNodeFsScheme fails if Orc support is disabled
> -----------------------------------------------------------------
>
>                 Key: IMPALA-8810
>                 URL: https://issues.apache.org/jira/browse/IMPALA-8810
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Frontend
>    Affects Versions: Impala 3.3.0
>            Reporter: Csaba Ringhofer
>            Priority: Major
>
> The problem is caused by going into this block when Orc support is disabled: 
> https://github.com/apache/impala/blob/51f30a6b9088cf3a0217a49a5792f1b41a8ad51b/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java#L416
> + that ExplainTest only mocks HdfsPartition.getFileFormat() but not 
> HdfsPartition.part.getInputFormatDescriptor().



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org

Reply via email to