yuluo-yx opened a new issue, #10859:
URL: https://github.com/apache/rocketmq/issues/10859

   ## Runtime platform environment
   macOS; reproduced with a deterministic unit test in the `tieredstore` module.
   
   ## RocketMQ version
   Branch: `develop`
   Git commit: `fd0c95920e0deac96ce2ae27442747cc5e65e930`
   
   ## JDK Version
   Eclipse Temurin 17.0.19+10
   
   ## Describe the Bug
   `IndexStoreService.putKey` retries an index write three times. If every 
attempt fails, it logs an error but returns `AppendResult.SUCCESS`, so callers 
can treat an index entry that was never stored as successful.
   
   ## Steps to Reproduce
   1. Set `currentWriteFile` to an `IndexFile` whose `putKey` returns 
`FILE_FULL`.
   2. Keep file rotation from replacing that test file.
   3. Call `IndexStoreService.putKey`.
   4. Observe three failed attempts followed by a `SUCCESS` result.
   
   ## What Did You Expect to See?
   The method should return the final failed `AppendResult` after all retries 
are exhausted.
   
   ## What Did You See Instead?
   The method returns `SUCCESS` despite all three writes returning `FILE_FULL`.
   
   ## Additional Context
   The normal success path and the existing three-attempt retry behavior do not 
need to change.


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