[
https://issues.apache.org/jira/browse/TAJO-1850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14964341#comment-14964341
]
ASF GitHub Bot commented on TAJO-1850:
--------------------------------------
Github user hyunsik commented on the pull request:
https://github.com/apache/tajo/pull/818#issuecomment-149390508
The patch looks good to me. I manually tested it. It passed all unit tests.
Could you remove TUtil::newHash method too?
> Using TUtil.newHash(Set/Map) should be replaced by Java's diamond operator
> --------------------------------------------------------------------------
>
> Key: TAJO-1850
> URL: https://issues.apache.org/jira/browse/TAJO-1850
> Project: Tajo
> Issue Type: Improvement
> Reporter: Hyunsik Choi
> Assignee: Dongkyu Hwangbo
> Priority: Trivial
> Labels: newbie
>
> See the title. We introduced Java 7. We don't need TUtil.newHash(Set|Map)
> utility methods anymore. We need to eliminate them.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
