wuYin commented on a change in pull request #394:
URL: https://github.com/apache/pulsar-client-go/pull/394#discussion_r528032340



##########
File path: pulsar/producer_partition.go
##########
@@ -553,9 +613,7 @@ func (p *partitionProducer) ReceivedSendReceipt(response 
*pb.CommandSendReceipt)
        // lock the pending item while sending the requests
        pi.Lock()
        defer pi.Unlock()
-       if pi.sentAt > 0 {

Review comment:
       @cckellogg 
   I used `sentAt` for checking timeout of messasges.
   `sentAt` always set before puting into pending queue, so I removed this 
predicate.
   Like 
[sendRequest.publishTime](https://github.com/apache/pulsar-client-go/blob/master/pulsar/producer_partition.go#L565)
 did.
   




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


Reply via email to