You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tajo.apache.org by "Jihoon Son (JIRA)" <ji...@apache.org> on 2013/10/26 06:40:30 UTC

[jira] [Commented] (TAJO-286) Refactor TableDesc, TableMeta, and Fragment

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

Jihoon Son commented on TAJO-286:
---------------------------------

+1 for this issue.
I'll review the patch at tonight. 

> 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)