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

Rajesh Balamohan commented on TEZ-3192:
---------------------------------------

+1. lgtm. Thanks [~jeagles]

> IFile#checkState creating unnecessary objects though auto-boxing
> ----------------------------------------------------------------
>
>                 Key: TEZ-3192
>                 URL: https://issues.apache.org/jira/browse/TEZ-3192
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-3192.1.patch
>
>
> checkState is a varargs function which takes Objects. ints and longs create 
> unnecessary Integers and Long objects through Integer.valueOf and 
> Long.valueOf. This is used in the read key and read value loop so while 
> small, puts this on par with the MR equivalent.



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

Reply via email to