[ 
https://issues.apache.org/jira/browse/PHOENIX-1311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15231776#comment-15231776
 ] 

ASF GitHub Bot commented on PHOENIX-1311:
-----------------------------------------

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

    https://github.com/apache/phoenix/pull/153#discussion_r58988872
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/hadoop/hbase/ipc/controller/MetadataRpcController.java
 ---
    @@ -36,7 +37,16 @@
                        .add(PhoenixDatabaseMetaData.SYSTEM_CATALOG_NAME)
                        .add(PhoenixDatabaseMetaData.SYSTEM_STATS_NAME)
                        .add(PhoenixDatabaseMetaData.SYSTEM_SEQUENCE_NAME)
    -                   
.add(PhoenixDatabaseMetaData.SYSTEM_FUNCTION_NAME).build();
    +                   .add(PhoenixDatabaseMetaData.SYSTEM_FUNCTION_NAME)
    +            
.add(SchemaUtil.getPhysicalTableName(PhoenixDatabaseMetaData.SYSTEM_CATALOG_NAME_BYTES,
 true)
    --- End diff --
    
    is the namespace mapping enabled always for system tables (which is what I 
am inferring from true being passed from isNamespaceMapped) ? Looking at 
QueryServicesOptions.java, looks like it isn't.


> HBase namespaces surfaced in phoenix
> ------------------------------------
>
>                 Key: PHOENIX-1311
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1311
>             Project: Phoenix
>          Issue Type: New Feature
>            Reporter: nicolas maillard
>            Assignee: Ankit Singhal
>            Priority: Minor
>             Fix For: 4.8.0
>
>         Attachments: PHOENIX-1311.docx, PHOENIX-1311_v1.patch, 
> PHOENIX-1311_v2.patch, PHOENIX-1311_wip.patch, PHOENIX-1311_wip_2.patch
>
>
> Hbase (HBASE-8015) has the concept of namespaces in the form of 
> myNamespace:MyTable it would be great if Phoenix leveraged this feature to 
> give a database like feature on top of the table.
> Maybe to stay close to Hbase it could also be a create DB:Table...
> or DB.Table which is a more standard annotation?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to