accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ed Coleman (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3329) Consider consolidation of "timing" classes
Date Fri, 27 Mar 2015 10:15:52 GMT


Ed Coleman commented on ACCUMULO-3329:

OpTimer is one of the classes that explicitly sets the logging level. I was going to look
into re-factoring OpTimer so that it would not have a direct dependency on log4j.

I will keep this ticket in mind and see if maybe we can get rid of OpTimer altogether by using
one of the other suggestions.

> Consider consolidation of "timing" classes
> ------------------------------------------
>                 Key: ACCUMULO-3329
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client, master, tserver
>            Reporter: Josh Elser
>              Labels: newbie
>             Fix For: 1.7.0
> We have a number of "timing" classes in or used by the codebase
> * org.apache.accumulo.core.util.StopWatch
> * org.apache.accumulo.core.util.OpTimer
> * Traces
> * Guava's Stopwatch
> I'm assuming that consolidation of all of the timings into Traces would be the best (assuming
that if we care about the timing of a given operation implies that we would also care about
the timing of the "bigger picture").
> If we can remove some of our timer classes, that would be great. Not suggesting that
we forcibly prevent the use of Stopwatches/TImers in the codebase entirely -- just where it
makes sense.

This message was sent by Atlassian JIRA

View raw message