[phoenix] branch 5.1 updated: PHOENIX-6514 Exception should be thrown

[st...@apache.org]

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

stoty pushed a commit to branch 5.1
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/5.1 by this push:
     new 0810be3  PHOENIX-6514 Exception should be thrown
0810be3 is described below

commit 0810be324da31ae20c377be7c336a3efb1fef3f2
Author: Wei-Chiu Chuang <we...@apache.org>
AuthorDate: Wed Jul 14 11:17:43 2021 +0800

    PHOENIX-6514 Exception should be thrown
    
    Change-Id: I626eb3878a29e1467835c48f8f446edc72ed04c8
---
 .../java/org/apache/phoenix/coprocessor/PhoenixAccessController.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/PhoenixAccessController.java b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/PhoenixAccessController.java
index 18787ec..e15ac43 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/PhoenixAccessController.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/PhoenixAccessController.java
@@ -285,7 +285,7 @@ public class PhoenixAccessController extends BaseMetaDataEndpointObserver {
                     AccessControlClient.grant(conn, TableName.valueOf(table), toUser , null, null,
                             actions);
                 } catch (Throwable e) {
-                    new DoNotRetryIOException(e);
+                    throw new DoNotRetryIOException(e);
                 }
                 return null;
             }