[
https://issues.apache.org/jira/browse/TAJO-1735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14653228#comment-14653228
]
ASF GitHub Bot commented on TAJO-1735:
--------------------------------------
Github user blrunner commented on a diff in the pull request:
https://github.com/apache/tajo/pull/673#discussion_r36164948
--- 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 --
Why don't you implement a constructor which needs only MetadataProvider?
> 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)
