You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/07/27 00:59:36 UTC

[GitHub] [pulsar] Lagranmoon opened a new issue #7669: stats recorder time unit error

Lagranmoon opened a new issue #7669:
URL: https://github.com/apache/pulsar/issues/7669


   **Describe the bug**
   pulsar client use info level log to record throughput and latency in class org.apache.pulsar.client.impl.ProducerStatsRecorderImpl, but the latency was divided by 1000 by mistake,the ds was already in ms, so there is no need to convert time,it will cause a lot of misunderstanding when testing the producer latency.
   ![image](https://user-images.githubusercontent.com/24788692/88494152-40d02800-cfe7-11ea-8f2e-fcb023a8cee7.png)
   
   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [pulsar] Lagranmoon closed issue #7669: stats recorder time unit error

Posted by GitBox <gi...@apache.org>.
Lagranmoon closed issue #7669:
URL: https://github.com/apache/pulsar/issues/7669


   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [pulsar] zymap commented on issue #7669: stats recorder time unit error

Posted by GitBox <gi...@apache.org>.
zymap commented on issue #7669:
URL: https://github.com/apache/pulsar/issues/7669#issuecomment-697291433


   We can close it because it has been fixed.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [pulsar] zymap commented on issue #7669: stats recorder time unit error

Posted by GitBox <gi...@apache.org>.
zymap commented on issue #7669:
URL: https://github.com/apache/pulsar/issues/7669#issuecomment-697291527


   @codelipenghui 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [pulsar] codelipenghui commented on issue #7669: stats recorder time unit error

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on issue #7669:
URL: https://github.com/apache/pulsar/issues/7669#issuecomment-664070472


   Are you interested in fixing it?


----------------------------------------------------------------
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:
users@infra.apache.org