activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From franz1981 <...@git.apache.org>
Subject [GitHub] activemq-artemis issue #1605: ARTEMIS-1476 HdrHistogram support on verbose S...
Date Wed, 25 Oct 2017 19:56:26 GMT
Github user franz1981 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1605
  
    @clebertsuconic 
    > if this complexity was added around say.. for instance... JournalImpl::show statistics..
I would go for it... but here at the sync calculation? I think this is being a lot of complexity
for something that is supposed to be simple.
    
    +100 This is exactly the point mate :)
    So do you think that is better to provide an ad hoc command (if not exists) for that (ie
single write latencies measurements)?
    This time I have trying to avoid having 2 tools similar but different, but I trust you
if you see a different usage for it...
    
    >for instance.. you could have chosen to add the Histogram without making any other
changes
    This is sadly seems not possible (to me at least): each write request (with ASYNCIO and
not syncWrites) need its own start time in order to compute the latency of the operation (and
record it on the histogram).
    How can you provide for each write its own start time with ASYNCIO and not syncWrites?
    



---

Mime
View raw message