[
https://issues.apache.org/jira/browse/MAPREDUCE-2500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13034463#comment-13034463
]
Luke Lu commented on MAPREDUCE-2500:
------------------------------------
Minor nits:
# The ctor lookup code probably should save the lookup result instead make
another cache.get call.
# You can probably get rid of the null in clazz.getConstructor(null) in
RecordFactoryPBImpl to avoid the vararg warning.
> MR 279: PB factories are not thread safe
> ----------------------------------------
>
> Key: MAPREDUCE-2500
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-2500
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Components: mrv2
> Reporter: Siddharth Seth
> Assignee: Siddharth Seth
> Fix For: 0.23.0
>
> Attachments: MR2500.patch, MR2500_2.patch
>
>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira