codelipenghui commented on a change in pull request #5786: Fix
ArrayIndexOutOfBoundsException in PerformanceProducer.
URL: https://github.com/apache/pulsar/pull/5786#discussion_r352929587
##########
File path:
pulsar-testclient/src/main/java/org/apache/pulsar/testclient/PerformanceProducer.java
##########
@@ -87,8 +87,8 @@
private static final LongAdder totalMessagesSent = new LongAdder();
private static final LongAdder totalBytesSent = new LongAdder();
- private static Recorder recorder = new
Recorder(TimeUnit.SECONDS.toMillis(120000), 5);
- private static Recorder cumulativeRecorder = new
Recorder(TimeUnit.SECONDS.toMillis(120000), 5);
+ private static Recorder recorder = new
Recorder(TimeUnit.SECONDS.toMicros(120000), 5);
Review comment:
Currently, the microseconds was recorded into recorder but output in mills.
We can change the record value to mills or change the highestTrackableValue to
microseconds. The latter is currently selected
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
