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

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

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

    https://github.com/apache/tajo/pull/671#discussion_r39266733
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/SeqScanExec.java
 ---
    @@ -198,7 +201,34 @@ public void init() throws IOException {
           // for non-projected fields.
           Schema actualInSchema = scanner.isProjectable() ? projectedFields : 
inSchema;
     
    -      this.projector = new Projector(context, actualInSchema, outSchema, 
plan.getTargets());
    +      Target[] realTargets;
    --- End diff --
    
    It would be better if this change is extracted to one separate method like 
``initializeProjector``.


> Change the default output file format.
> --------------------------------------
>
>                 Key: TAJO-1340
>                 URL: https://issues.apache.org/jira/browse/TAJO-1340
>             Project: Tajo
>          Issue Type: Improvement
>          Components: Java Client, JDBC Driver, Offheap, Storage
>            Reporter: Hyunsik Choi
>            Assignee: Jinho Kim
>             Fix For: 0.11.0, 0.12.0
>
>         Attachments: TAJO-1340.patch, TAJO-1340_2.patch
>
>
> 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