Github user ankitsinghal commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/283#discussion_r152481907
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/PhoenixAccessController.java
 ---
    @@ -229,17 +227,12 @@ public void 
handleRequireAccessOnDependentTable(String request, String userName,
                         + dependentTable);
                 return;
             }
    -        if (isAutomaticGrantEnabled) {
    --- End diff --
    
    I think , we can keep this option. as it will help admin if he doesn't want 
to see the permissions problems on the dependent table and wants to manage the 
permission by himself once the table is created(with downtime) and don't want 
to rely on the automatic grant.


---

Reply via email to