Repository: drill Updated Branches: refs/heads/master 27aff35b5 -> ef0fafea2
DRILL-3958: Return a valid error message when storage plugin fails This closes #1067 Project: http://git-wip-us.apache.org/repos/asf/drill/repo Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/bf56cd91 Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/bf56cd91 Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/bf56cd91 Branch: refs/heads/master Commit: bf56cd917bd8f054794489e2b252d79c039d0286 Parents: 27aff35 Author: Rahul Raj <[email protected]> Authored: Tue Dec 12 14:21:53 2017 +0530 Committer: Parth Chandra <[email protected]> Committed: Thu Jan 11 16:55:44 2018 -0800 ---------------------------------------------------------------------- .../java/org/apache/drill/exec/server/rest/StorageResources.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/drill/blob/bf56cd91/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java index cf5eb57..ca10860 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java @@ -162,8 +162,8 @@ public class StorageResources { plugin.createOrUpdateInStorage(storage); return message("success"); } catch (ExecutionSetupException e) { - logger.debug("Unable to create/ update plugin: " + plugin.getName()); - return message("error (unable to create/ update storage)"); + logger.error("Unable to create/ update plugin: " + plugin.getName(), e); + return message("Error while creating/ updating storage : " + (e.getCause() != null ? e.getCause().getMessage() : e.getMessage())); } }
