[
https://issues.apache.org/jira/browse/TAJO-1450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14568342#comment-14568342
]
ASF GitHub Bot commented on TAJO-1450:
--------------------------------------
Github user jinossy commented on a diff in the pull request:
https://github.com/apache/tajo/pull/596#discussion_r31486521
--- Diff: tajo-common/src/main/java/org/apache/tajo/storage/NullTuple.java
---
@@ -117,6 +124,11 @@ public char getChar(int fieldId) {
}
@Override
+ public byte[] getTextBytes(int fieldId) {
+ return new byte[0];
--- End diff --
Can you change to static variable ?
> 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)
