You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by js...@apache.org on 2016/01/28 23:29:32 UTC

[2/2] drill git commit: DRILL-4322: Add underlying exception message when IOException causes DROP TABLE failure

DRILL-4322: Add underlying exception message when IOException causes DROP TABLE failure

This closes #344


Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/1b51850f
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/1b51850f
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/1b51850f

Branch: refs/heads/2653-merge
Commit: 1b51850f31c02f0a7fa77f0258a83081a9d9e826
Parents: 9fa7b28
Author: Jason Altekruse <al...@gmail.com>
Authored: Thu Jan 28 11:18:43 2016 -0800
Committer: Jason Altekruse <al...@gmail.com>
Committed: Thu Jan 28 14:13:56 2016 -0800

----------------------------------------------------------------------
 .../apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/drill/blob/1b51850f/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
----------------------------------------------------------------------
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java b/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
index 65e387e..1623463 100644
--- a/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
+++ b/exec/java-exec/src/main/java/org/apache/drill/exec/store/dfs/WorkspaceSchemaFactory.java
@@ -728,13 +728,13 @@ public class WorkspaceSchemaFactory {
         fs.delete(new Path(defaultLocation, tableRename), true);
       } catch (AccessControlException e) {
         throw UserException
-            .permissionError()
-            .message("Unauthorized to drop table", e)
+            .permissionError(e)
+            .message("Unauthorized to drop table")
             .build(logger);
       } catch (IOException e) {
         throw UserException
-            .dataWriteError()
-            .message("Failed to drop table", e)
+            .dataWriteError(e)
+            .message("Failed to drop table: " + e.getMessage())
             .build(logger);
       }
     }