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

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_r57371949
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java
 ---
    @@ -1115,6 +1212,24 @@ private PFunction 
loadFunction(RegionCoprocessorEnvironment env, byte[] key,
                 return null;
             }
     
    +    private PSchema loadSchema(RegionCoprocessorEnvironment env, byte[] 
key, ImmutableBytesPtr cacheKey,
    +            long clientTimeStamp, long asOfTimeStamp) throws IOException, 
SQLException {
    +        Region region = env.getRegion();
    +        Cache<ImmutableBytesPtr, PMetaDataEntity> metaDataCache = 
GlobalCache.getInstance(this.env).getMetaDataCache();
    +        PSchema schema = (PSchema)metaDataCache.getIfPresent(cacheKey);
    +        // We always cache the latest version - fault in if not in cache
    +        if (schema != null) { return schema; }
    +        ArrayList<byte[]> arrayList = new ArrayList<byte[]>(1);
    +        arrayList.add(key);
    +        List<PSchema> schemas = buildSchemas(arrayList, region, 
asOfTimeStamp, cacheKey);
    +        if (schemas != null) return schemas.get(0);
    +        // if not found then check if newer table already exists and add 
delete marker for timestamp
    --- End diff --
    
    Change comment to mention schema and not table.


> 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