dazengelead opened a new issue, #7386:
URL: https://github.com/apache/rocketmq/issues/7386

   ### Before Creating the Bug Report
   
   - [X] I found a bug, not just asking a question, which should be created in 
[GitHub Discussions](https://github.com/apache/rocketmq/discussions).
   
   - [X] I have searched the [GitHub 
Issues](https://github.com/apache/rocketmq/issues) and [GitHub 
Discussions](https://github.com/apache/rocketmq/discussions)  of this 
repository and believe that this is not a duplicate.
   
   - [X] I have confirmed that this bug belongs to the current repository, not 
other repositories of RocketMQ.
   
   
   ### Runtime platform environment
   
   rocketmq 5.1.3 
   
   ### RocketMQ version
   
   rocketmq 5.1.3 
   
   ### JDK Version
   
   _No response_
   
   ### Describe the Bug
   
   
   org.apache.rocketmq.client.impl.consumer.ProcessQueue#cleanExpiredMsg
   bug in line: 90
   `
   if (StringUtils.isNotEmpty(consumeStartTimeStamp) && 
System.currentTimeMillis() - Long.parseLong(consumeStartTimeStamp) > 
pushConsumer.getConsumeTimeout() * 60 * 1000) {
                               msg = msgTreeMap.firstEntry().getValue();
    }
   `
   
   ### Steps to Reproduce
   
   
   org.apache.rocketmq.client.impl.consumer.ProcessQueue#cleanExpiredMsg
   bug in line: 90
   `
   if (StringUtils.isNotEmpty(consumeStartTimeStamp) && 
System.currentTimeMillis() - Long.parseLong(consumeStartTimeStamp) > 
pushConsumer.getConsumeTimeout() * 60 * 1000) {
                               msg = msgTreeMap.firstEntry().getValue();
    }
   `
   
   ### What Did You Expect to See?
   
   org.apache.rocketmq.client.impl.consumer.ProcessQueue#cleanExpiredMsg
   bug in line: 90
   
   condition should be:  StringUtils.isNotEmpty(consumeStartTimeStamp) && 
(System.currentTimeMillis() - Long.parseLong(consumeStartTimeStamp)) > 
pushConsumer.getConsumeTimeout() * 60 * 1000
   
   ### What Did You See Instead?
   
   org.apache.rocketmq.client.impl.consumer.ProcessQueue#cleanExpiredMsg
   bug in line: 90
   
   condition should be:  StringUtils.isNotEmpty(consumeStartTimeStamp) && 
(System.currentTimeMillis() - Long.parseLong(consumeStartTimeStamp)) > 
pushConsumer.getConsumeTimeout() * 60 * 1000
   
   ### Additional Context
   
   _No response_


-- 
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-unsubscr...@rocketmq.apache.org.apache.org

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

Reply via email to