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

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

GitHub user jihoonson opened a pull request:

    https://github.com/apache/tajo/pull/596

    TAJO-1450: TAJO-1450 Encapsulate Datum in Tuple

    Just refactored patch of @navis' work in #465.
    His patch looks good to me. If any other objections, I'll commit.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jihoonson/tajo-2 TAJO-1450

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tajo/pull/596.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #596
    
----
commit 358b0aa313090653ea200b4e08aed975f7e16100
Author: navis.ryu <[email protected]>
Date:   2015-03-26T14:19:52Z

    TAJO-1450 Encapsulate Datum in Tuple

commit 02a06786fc8e8fac693e47dbe6a20e948d96a7da
Author: Jihoon Son <[email protected]>
Date:   2015-04-28T10:45:14Z

    Merge branch 'TAJO-1450' of https://github.com/navis/tajo into TAJO-1450

commit bd6774e00a7a6436d86d976b0b713e24b74dcbe5
Author: Jihoon Son <[email protected]>
Date:   2015-06-01T08:17:40Z

    Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into 
TAJO-1450
    
    Conflicts:
        tajo-client/src/main/java/org/apache/tajo/storage/RowStoreUtil.java
        tajo-common/src/main/java/org/apache/tajo/storage/EmptyTuple.java
        tajo-common/src/main/java/org/apache/tajo/storage/Tuple.java
        
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/HashFullOuterJoinExec.java
        
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/HashJoinExec.java
        
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/HashLeftOuterJoinExec.java
        
tajo-core/src/main/java/org/apache/tajo/master/exec/NonForwardQueryResultSystemScanner.java
        
tajo-core/src/test/java/org/apache/tajo/engine/planner/global/TestBroadcastJoinPlan.java
        
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestHashSemiJoinExec.java
        
tajo-core/src/test/java/org/apache/tajo/engine/planner/physical/TestLeftOuterNLJoinExec.java
        
tajo-plan/src/main/java/org/apache/tajo/plan/function/stream/CSVLineSerializer.java
        
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/RowStoreUtil.java
        
tajo-storage/tajo-storage-common/src/main/java/org/apache/tajo/storage/TextSerializerDeserializer.java
        
tajo-storage/tajo-storage-hbase/src/main/java/org/apache/tajo/storage/hbase/HBasePutAppender.java
        
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/parquet/TajoRecordConverter.java
        
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/parquet/TajoWriteSupport.java
        
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java
        
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/text/CSVLineDeserializer.java
        
tajo-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/text/TextFieldSerializerDeserializer.java
        
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestMergeScanner.java
        
tajo-storage/tajo-storage-hdfs/src/test/java/org/apache/tajo/storage/TestStorages.java

commit f1ab22157d7e1dfad31f1493f40963f03428c69c
Author: Jihoon Son <[email protected]>
Date:   2015-06-01T08:28:41Z

    TAJO-1450

commit 242e4ab022e591942ad2696a122bed26a7e5ae2a
Author: Jihoon Son <[email protected]>
Date:   2015-06-01T12:44:52Z

    TAJO-1450

commit 1ed6047ee9db7e80b275fc294d221dad1e36f954
Author: Jihoon Son <[email protected]>
Date:   2015-06-01T13:48:16Z

    TAJO-1450

----


> Encapsulate Datum in Tuple
> --------------------------
>
>                 Key: TAJO-1450
>                 URL: https://issues.apache.org/jira/browse/TAJO-1450
>             Project: Tajo
>          Issue Type: Task
>            Reporter: Navis
>            Assignee: Navis
>            Priority: Minor
>             Fix For: 0.11.0
>
>
> Currently, Tuple exposes Datum inside of it but it seemed not necessary(, 
> which is not fully confirmed). If we remove Datum returning APIs from Tuple, 
> Tuple can be implemented more effective and free. I think this can be a 
> subtask of vectorization.



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

Reply via email to