[
https://issues.apache.org/jira/browse/MAPREDUCE-6005?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14080524#comment-14080524
]
Binglin Chang commented on MAPREDUCE-6005:
------------------------------------------
Hi Sean,
bq. About the leak, at src / main / native / src / lib / Merge.cc, there is a
similar mem leak, you only fixed the leak in PartitionBucketIterator.cc
In merger, all MergeEntryPtr is owned by Merger::_entries, and is deleted in
~Merger at end, so it doesn't require additional care.
bq. maybe better use snsprintf(buf_ptr, "%02X", ...)
Sure, I will update the patch
> native-task: fix some valgrind errors
> --------------------------------------
>
> Key: MAPREDUCE-6005
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6005
> Project: Hadoop Map/Reduce
> Issue Type: Sub-task
> Components: task
> Reporter: Binglin Chang
> Assignee: Binglin Chang
> Attachments: MAPREDUCE-6005.v1.patch, MAPREDUCE-6005.v2.patch
>
>
> Running test with valgrind shows there are some bugs, this jira try to fix
> them.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
