[
https://issues.apache.org/jira/browse/TAJO-1207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14259317#comment-14259317
]
ASF GitHub Bot commented on TAJO-1207:
--------------------------------------
Github user charsyam closed the pull request at:
https://github.com/apache/tajo/pull/267
> Remove redundant code in Bytes.java in tajo-common
> --------------------------------------------------
>
> Key: TAJO-1207
> URL: https://issues.apache.org/jira/browse/TAJO-1207
> Project: Tajo
> Issue Type: Improvement
> Reporter: DaeMyung Kang
> Assignee: DaeMyung Kang
> Priority: Trivial
>
> in Bytes.java
> padHead and padTail
> Don't need to set array to 0.
> because java alloc byte array and initialize them as 0.
> ```java
> byte [] padding = new byte[length];
> //so below is redundant block
> for (int i = 0; i < length; i++) {
> padding[i] = 0;
> }
> return add(padding,a);
> ```
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
