tomscut opened a new pull request #13057:
URL: https://github.com/apache/pulsar/pull/13057


   
   ### Motivation
   
   *The time unit in this exception message is ns, which is not very readable. 
We can change it from ns to ms.*
   ```
   org.apache.pulsar.client.api.PulsarClientException$TimeoutException:
   The producer xxx can not send message to the topic xxx within given timeout 
: createdAt 461913074 ns ago, firstSentAt 29545553038276935 ns ago, lastSentAt 
29545553038276935 ns ago, retryCount 0 at 
org.apache.pulsar.client.api.PulsarClientException.unwrap(PulsarClientException.java:916)
   at 
org.apache.pulsar.client.impl.TypedMessageBuilderImpl.send(TypedMessageBuilderImpl.java:93)
   at org.apache.pulsar.client.impl.ProducerBase.send(ProducerBase.java:63)
   at 
com.yum.boh.oh.service.impl.StoreOrderPostServiceImpl.generalProcessing(StoreOrderPostServiceImpl.java:272)
   at 
com.yum.boh.oh.service.impl.StoreOrderPostServiceImpl.saveThirdOrder(StoreOrderPostServiceImpl.java:72)
   at 
com.yum.boh.oh.controller.StoreOrderController.postOrderInfo$original$T8425mfx(StoreOrderController.java:39)
   at 
com.yum.boh.oh.controller.StoreOrderController.postOrderInfo$original$T8425mfx$accessor$vJljNzML(StoreOrderController.java)
   at 
com.yum.boh.oh.controller.StoreOrderController$auxiliary$nysalhgy.call(Unknown 
Source)
   at 
org.apache.skywalking.apm.agent.core.plugin.interceptor.enhance.InstMethodsInter.intercept(InstMethodsInter.java:86)
   ...
   ```
   
   ### Modifications
   
   *Change the time units from ns to ms for ProducerImpl#OpSendMsg.*
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (no)
     - The public API: (no)
     - The schema: (no)
     - The default values of configurations: (no)
     - The wire protocol: (no)
     - The rest endpoints: (no)
     - The admin cli options: (no)
     - Anything that affects deployment: (no)
   
   ### Documentation
   
   Check the box below and label this PR (if you have committer privilege).
   
   Need to update docs? 
   - [ ] `doc-required` 
   - [x] `no-need-doc` 
   - [ ] `doc` 
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to