Sailesh Mukil has posted comments on this change.

Change subject: IMPALA-5331: Use new libHDFS API to address "Unknown Error 255"
......................................................................


Patch Set 1:

Questions for reviewers:

Do you think we should call hdfsGetLastExceptionRootCause() for all HDFS errors 
(and not only 255)? In some cases, there may be more useful information, while 
in most other cases, the error message will be redundant (eg: 'file not found' 
printed twice).

Also, does anyone know of a test case that deterministically reproduces an 
"Unknown Error 255"? I spent a few hours trying to recreate one, but none seem 
simple enough to automate into our test suite.

-- 
To view, visit http://gerrit.cloudera.org:8080/6894
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I181e316ed63b70b94d4f7a7557d398a931bb171d
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-HasComments: No

Reply via email to