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

ASF GitHub Bot commented on TAJO-1340:
--------------------------------------

Github user jihoonson commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/671#discussion_r36613977
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/engine/planner/global/GlobalPlanner.java
 ---
    @@ -163,7 +164,7 @@ public void build(QueryContext queryContext, MasterPlan 
masterPlan) throws IOExc
       private static void setFinalOutputChannel(DataChannel outputChannel, 
Schema outputSchema) {
         outputChannel.setShuffleType(NONE_SHUFFLE);
         outputChannel.setShuffleOutputNum(1);
    -    outputChannel.setStoreType("TEXT");
    +    outputChannel.setStoreType(BuiltinStorages.DRAW);
    --- End diff --
    
    How about adding a final variable for the default store type and using it 
in the class?


> Change the default output file format.
> --------------------------------------
>
>                 Key: TAJO-1340
>                 URL: https://issues.apache.org/jira/browse/TAJO-1340
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: Hyunsik Choi
>            Assignee: Jinho Kim
>             Fix For: 0.11.0
>
>
> Currently, the default output file is CSV. Due to its nature, CSV has mainly 
> three problems:
>  * Its line or field delimiter can be duplicated to some character included 
> in the result data.
>  * Plan text file is likely to be larger than other file formats.
>  * Its read and write performance is slow.
> We need to change the default output file format into other file formats. We 
> also need to investigate which file format is the best for it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to