[
https://issues.apache.org/jira/browse/TAJO-286?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13806226#comment-13806226
]
Jihoon Son commented on TAJO-286:
---------------------------------
+1 for the patch.
Please remove some commented tests in TestFragment before you submit.
> Refactor TableDesc, TableMeta, and Fragment
> -------------------------------------------
>
> Key: TAJO-286
> URL: https://issues.apache.org/jira/browse/TAJO-286
> Project: Tajo
> Issue Type: Improvement
> Components: catalog
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.8-incubating
>
> Attachments: PATCH-286.patch
>
>
> In the current implementation, TableDesc and TableMeta are implemented
> through interface and its implementation. This is unnecessary abstraction.
> For simplicity, this patch removes interfaces and merge them into the
> concrete classes.
> In addition, TableDesc and TableMeta's role is ambiguous. This patch
> clarifies their roles as follows:
> * TableMeta contains usual physical information which is used in workers.
> * TableDesc contains logical information about a table and others which are
> not used in workers.
> As a result, I've moved TableStats and Schema from TableMeta to TableDesc.
> Besides, in the current implementation, Fragment also is subclassed of
> TableDesc. But, this relationship is wrong. Fragment is independent one of
> TableDesc. This patch changes this relationship. They are independent ones.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
