[
https://issues.apache.org/jira/browse/TAJO-1151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14206468#comment-14206468
]
ASF GitHub Bot commented on TAJO-1151:
--------------------------------------
Github user jihoonson commented on a diff in the pull request:
https://github.com/apache/tajo/pull/228#discussion_r20152417
--- Diff:
tajo-storage/src/main/java/org/apache/tajo/storage/text/DelimitedTextFile.java
---
@@ -316,9 +316,10 @@ public void init() throws IOException {
try {
// we need to discuss the De/Serializer interface. so custom serde
is to disable
- String serdeClass =
this.meta.getOption(StorageConstants.TEXTFILE_SERDE,
+ /*String serdeClass =
this.meta.getOption(StorageConstants.TEXTFILE_SERDE,
--- End diff --
If we don't need this code line, please remove it.
> Implement the ByteBuffer-based De/Serializer
> ---------------------------------------------
>
> Key: TAJO-1151
> URL: https://issues.apache.org/jira/browse/TAJO-1151
> Project: Tajo
> Issue Type: Sub-task
> Components: storage
> Reporter: Jinho Kim
> Assignee: Jinho Kim
>
> We should add ByteBuffer-based De/Serializer instead of {{byte[]}}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
