hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Zhong (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MAPREDUCE-6069) native-task: Style fixups and dead code removal
Date Thu, 04 Sep 2014 06:43:52 GMT

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

Sean Zhong commented on MAPREDUCE-6069:
---------------------------------------

Todd,

I cannot express enough my thanks for this! I know cleaning this up will take huge amount
of time. So, it is an essential steps towards making it acceptable by the community.

Consider this patch is huge, maybe it is more easy to break it down to several small patches,
so that we can track the changes step by step. We can work on your patch to break it down
to several smaller ones.

> native-task: Style fixups and dead code removal
> -----------------------------------------------
>
>                 Key: MAPREDUCE-6069
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6069
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: task
>            Reporter: Todd Lipcon
>         Attachments: mr-6069.txt
>
>
> A few more cleanup things we should address:
> - fix style issues (eg lines too long, bad indentation, commented-out code blocks, etc)
both in Java and C++
> - Found a few pieces of unused code by running a coverage tool. We should remove them



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

Mime
View raw message