[ 
https://issues.apache.org/jira/browse/SANDBOX-261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12657478#action_12657478
 ] 

Ken Tanaka commented on SANDBOX-261:
------------------------------------

This comment repeats what was mentioned on the development email list 
([email protected]), so the additional dialog is stored in jira with the 
patch. I compiled and ran the test code. The new code looks like it will be 
compatible with the previous code as well.

 - - -

Your new patch looks good, I would say that it should be committed to the 
project.

Thanks,
Ken

Matt Sanford wrote:
> Hi there,
>
>     I had forgotten to include the pom.xml. I added that change and made the 
> other one you mentioned as well.
>
> Thanks;
>   — Matt Sanford
>
> On Dec 16, 2008, at 11:49 AM, Ken Tanaka wrote:
>
>> Matt,
>>
>> What do you think of adding to the pom.xml
>>
>>   <dependency>
>>     <groupId>commons-math</groupId>
>>     <artifactId>commons-math</artifactId>
>>     <version>1.2</version>
>>     <optional>true</optional>
>>   </dependency>
>>
>> and using SynchronizedDescriptiveStatistics instead of 
>> DescriptiveStatisticsImpl, which is now deprecated (as of commons-math-1.2)?
>>
>> -Ken


> Add statistics gathering base stage
> -----------------------------------
>
>                 Key: SANDBOX-261
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-261
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: Pipeline
>    Affects Versions: Nightly Builds
>         Environment: N/A
>            Reporter: Matt Sanford
>             Fix For: Nightly Builds
>
>         Attachments: add_base_stage_with_stats_and_jmx.2.patch
>
>
> Create a base stage implementation that collects statistics common to most 
> (if not all) stages. Log these periodically and allow collection and 
> configuration via JMX. I have a patch for this I will attach to the ticket.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to