Repository: incubator-sentry Updated Branches: refs/heads/master 95b1e40e7 -> cb92ceb3b
SENTRY-1015: Improve Sentry + Hive error message when user has insufficient privileges (Hao Hao via Lenni Kuff) Change-Id: Ic10a06bade1fd3a00519a5abeba1a278ceae2c8e Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/cb92ceb3 Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/cb92ceb3 Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/cb92ceb3 Branch: refs/heads/master Commit: cb92ceb3bb45b100538642038567640797553a3d Parents: 95b1e40 Author: Lenni Kuff <lsk...@cloudera.com> Authored: Wed Jan 20 23:29:51 2016 -0800 Committer: Lenni Kuff <lsk...@cloudera.com> Committed: Wed Jan 20 23:29:51 2016 -0800 ---------------------------------------------------------------------- .../java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/cb92ceb3/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java ---------------------------------------------------------------------- diff --git a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java index 699b6b2..9c1eff7 100644 --- a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java +++ b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java @@ -400,7 +400,7 @@ public class HiveAuthzBindingHook extends AbstractSemanticAnalyzerHook { + "\n Required privileges for this query: " + permsRequired; String msgForConsole = HiveAuthzConf.HIVE_SENTRY_PRIVILEGE_ERROR_MESSAGE + "\n " - + e.getMessage(); + + e.getMessage()+ "\n The required privileges: " + permsRequired; // AuthorizationException is not a real exception, use the info level to record this. LOG.info(msgForLog); throw new SemanticException(msgForConsole, e);
