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

tdsilva pushed a commit to branch 4.14-HBase-1.2
in repository https://gitbox.apache.org/repos/asf/phoenix.git

commit 172149dcf30e66dabbb0ad5b887a060dd475079b
Author: Monani Mihir <monani.mi...@gmail.com>
AuthorDate: Fri Dec 14 16:20:17 2018 +0530

    PHOENIX-5070 NPE when upgrading Phoenix 4.13.0 to Phoenix 4.14.1 with 
hbase-1.x branch in secure setup
---
 .../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 18a293f..befa87d 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
@@ -406,7 +406,7 @@ public class PhoenixAccessController extends 
BaseMetaDataEndpointObserver {
                 final List<UserPermission> userPermissions = new 
ArrayList<UserPermission>();
                 try (Connection connection = 
ConnectionFactory.createConnection(env.getConfiguration())) {
                     // Merge permissions from all accessController 
coprocessors loaded in memory
-                    for (BaseMasterAndRegionObserver service : 
accessControllers) {
+                    for (BaseMasterAndRegionObserver service : 
getAccessControllers()) {
                         // Use AccessControlClient API's if the 
accessController is an instance of 
org.apache.hadoop.hbase.security.access.AccessController
                         if 
(service.getClass().getName().equals(org.apache.hadoop.hbase.security.access.AccessController.class.getName()))
 {
                             
userPermissions.addAll(AccessControlClient.getUserPermissions(connection, 
tableName.getNameAsString()));

Reply via email to