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

ASF GitHub Bot commented on TAJO-1735:
--------------------------------------

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

    https://github.com/apache/tajo/pull/673#discussion_r36169035
  
    --- Diff: 
tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/CatalogServer.java
 ---
    @@ -97,19 +103,17 @@
       public CatalogServer() throws IOException {
         super(CatalogServer.class.getName());
         this.handler = new CatalogProtocolHandler();
    +    this.linkedMetadataManager = new 
LinkedMetadataManager(Collections.EMPTY_LIST);
         this.builtingFuncs = new ArrayList<FunctionDesc>();
       }
     
    -  public CatalogServer(Collection<FunctionDesc> sqlFuncs) throws 
IOException {
    -    this();
    +  public CatalogServer(Set<MetadataProvider> metadataProviders, 
Collection<FunctionDesc> sqlFuncs) throws IOException {
    --- End diff --
    
    It is because the constructor is unnecessary.


> Implement MetadataProvider and LinkedMetadataManager
> ----------------------------------------------------
>
>                 Key: TAJO-1735
>                 URL: https://issues.apache.org/jira/browse/TAJO-1735
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: Catalog
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>             Fix For: 0.11.0
>
>
> This patch add a MetadataProvider interface and LinkedMetadataManager;
>  * MetadataProvider provides a way to access linked meta data store through 
> an unified API. 
>  * LinkedMetadataManager manages all MetadataProvider instances.



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

Reply via email to