On 11/1/07, Alexander Kolbasov <[EMAIL PROTECTED]> wrote:
> > SUMMARY
> >
> >         This fast-track enhances the DTrace utility to address an
> >         existing RFE[1] requesting an aggregating function to calculate
> >         standard deviation, similar to the current aggregating function
> >         for average.
>
> Is it going to treat the aggregated values as uint64_t or int64_t? See the
> discussion for max()/min()/avg() above.
>

My initial take on this is that, if we're going to eventually fix that
for max()/min()/avg(), I should go ahead and implement this correctly
from the start.

My only concern with this would be the possibility for confusion if
stddev() is implemented this way before the fix for 6624541 goes in
(e.g., someone sees a mean, standard deviation and distribution that
just don't seem to match.)  I don't know how likely that would be in
practice, but the obvious solution is to get a fix for 6624541 in
before this goes in.  (I'd be happy to implement the fix for 6624541.)

Chad
_______________________________________________
dtrace-discuss mailing list
[email protected]

Reply via email to