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

steven zhang commented on KYLIN-904:
------------------------------------

You are right  scan recursively is necessary to local file system 
i think ResourceStore have a abstract method named collectResource
FileResourceStore and HBaseResourceStore have implementation respectively one 
scan recursively another scan plain  in the future

> Recursive queries to method collectResourceRecursively(ResourceStore.java) is 
> not necessary
> -------------------------------------------------------------------------------------------
>
>                 Key: KYLIN-904
>                 URL: https://issues.apache.org/jira/browse/KYLIN-904
>             Project: Kylin
>          Issue Type: Improvement
>    Affects Versions: 0.7.1
>            Reporter: steven zhang
>            Priority: Minor
>              Labels: github-import
>             Fix For: 0.7.1
>
>
> it is not necessary collect Resource Recursively when kylin load metadata to 
> cache 
> because the path of Resource in table  kylin_metadata such as /cube , 
> /cube_desc etc. is not prefix structure as follow example
> /a/b
> /a/b/xx
> /a/b/xx/xx
> /a/b/yy/yy
> /a/c
> even it have the prefix structure also it is unnecessary
> for example it will get all the records with prefix ‘a/b’ when query ‘a/b’
> /a/b
> /a/b/xx
> /a/b/xx/xx
> /a/b/yy/yy
> when query ’/a/b/xx‘ it will return children of it
> /a/b/xx/xx (it already queried before)
> so it is not necessary to query path of resource recursively



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

Reply via email to