[
https://issues.apache.org/jira/browse/MAPREDUCE-1762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12895210#action_12895210
]
Amar Kamat commented on MAPREDUCE-1762:
---------------------------------------
Scott/Chris,
I see the following piece of code in this patch
{code}
/**
* Set this counter by the given value
* @param the value to set
*/
public synchronized void setValue(long value) {
this.value = value;
}
{code}
Looks like the parameter name 'value' is missing from the Javadoc (i.e @param).
This is resulting in {{test-patch}} failure on trunk. Can one of you look into
this?
> Add a setValue() method in Counter
> ----------------------------------
>
> Key: MAPREDUCE-1762
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-1762
> Project: Hadoop Map/Reduce
> Issue Type: Improvement
> Affects Versions: 0.22.0
> Reporter: Scott Chen
> Assignee: Scott Chen
> Fix For: 0.22.0
>
> Attachments: M1762-2.patch, MAPREDUCE-1762.1.txt, MAPREDUCE-1762.txt
>
>
> Counters are very useful because of the logging and transmitting are already
> there.
> It is very convenient to transmit and store numbers. But currently Counter
> only has an increment() method.
> It will be nice if there can be a setValue() method in this class that will
> allow us to transmit wider variety of information through it.
> What do you think?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
