[drill] branch master updated: DRILL-7552: Add Helpful Error Message on Storage Plugin Creation/Update

[cg...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cgivre pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git


The following commit(s) were added to refs/heads/master by this push:
     new 18b790f  DRILL-7552: Add Helpful Error Message on Storage Plugin Creation/Update
18b790f is described below

commit 18b790f295eb1133916198dff28bc892ff9400e8
Author: luoc <lu...@163.com>
AuthorDate: Tue Dec 15 19:49:13 2020 +0800

    DRILL-7552: Add Helpful Error Message on Storage Plugin Creation/Update
---
 .../java/org/apache/drill/exec/store/StoragePluginRegistryImpl.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/store/StoragePluginRegistryImpl.java b/exec/java-exec/src/main/java/org/apache/drill/exec/store/StoragePluginRegistryImpl.java
index 6ca2732..390a32c 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/store/StoragePluginRegistryImpl.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/store/StoragePluginRegistryImpl.java
@@ -420,9 +420,13 @@ public class StoragePluginRegistryImpl implements StoragePluginRegistry {
       PluginHandle entry = restoreFromEphemeral(name, config);
       try {
         entry.plugin();
+      } catch (UserException e) {
+        // Provide helpful error messages.
+        throw new PluginException(e.getOriginalMessage(), e);
       } catch (Exception e) {
         throw new PluginException(String.format(
-            "Invalid plugin config for '%s'", name), e);
+            "Invalid plugin config for '%s', "
+          + "Please switch to Logs panel from the UI then check the log.", name), e);
       }
       oldEntry = pluginCache.put(entry);
     } else {