[GitHub] [phoenix] ss77892 commented on a change in pull request #1340: PHOENIX-6579 ACL check doesn't honor the namespace mapping for mapped…

[GitBox <gi...@apache.org>]

ss77892 commented on a change in pull request #1340:
URL: https://github.com/apache/phoenix/pull/1340#discussion_r781761088



##########
File path: phoenix-core/src/it/java/org/apache/phoenix/end2end/PermissionNSEnabledIT.java
##########
@@ -48,6 +56,43 @@ public PermissionNSEnabledIT() throws Exception {
     public static synchronized void doSetup() throws Exception {
         BasePermissionsIT.initCluster(true);
     }
+    private AccessTestAction createMappedView(final String schemaName, final String tableName) throws SQLException {

Review comment:
       Well, didn't realize it from the beginning, but with NS disabled this test case doesn't make sense. We have a single namespace 'default' and all system and user tables are in there. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org