Hello Matthew Jacobs,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/4125

to look at the new patch set (#5).

Change subject: IMPALA-3949: Log the error message in 
FileSystemUtil.copyToLocal()
......................................................................

IMPALA-3949: Log the error message in FileSystemUtil.copyToLocal()

Currently the code swallows the actual exception incase the call
to copy fails. To improve supportability, this commit returns the
the error stack trace to the caller for logging.

Change-Id: I5664a75aa837951de1d5dcc90e43bd8f313736b8
---
M fe/src/main/java/com/cloudera/impala/catalog/CatalogServiceCatalog.java
M fe/src/main/java/com/cloudera/impala/common/FileSystemUtil.java
2 files changed, 16 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/4125/5
-- 
To view, visit http://gerrit.cloudera.org:8080/4125
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5664a75aa837951de1d5dcc90e43bd8f313736b8
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Brock Noland <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>

Reply via email to