tajo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hyunsik Choi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TAJO-424) Make serializer/deserializer configurable in CSVFile
Date Mon, 16 Dec 2013 17:47:07 GMT

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

Hyunsik Choi commented on TAJO-424:
-----------------------------------

+1

Nice work. In addition, I would like to two suggestions. (1) SerializeDeserialize is verb,
and I think it is only verb in many member variables. How about changing the name to SerializeDeserialize*r*.
(2) In addition, if we support custom (de) serializer, CSVFile is not a proper name anymore.
We need to rename the format name and its properties, such csvfile.delimiter and csvfile.serde.
Probably, it is right time to rename them because Tajo is under the relatively early stage.
It would be good to create a separate jira issue for it.

Anyway, I agreed this patch.

> Make serializer/deserializer configurable in CSVFile
> ----------------------------------------------------
>
>                 Key: TAJO-424
>                 URL: https://issues.apache.org/jira/browse/TAJO-424
>             Project: Tajo
>          Issue Type: Improvement
>          Components: storage
>            Reporter: Jinho Kim
>            Assignee: Jinho Kim
>         Attachments: TAJO-424.patch
>
>
> The CSVFile serializer/deserializer is fixed to TextSerializeDeserialize in the LazyTuple.
This should be configurable.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Mime
View raw message