andyhuangdev opened a new pull request, #11263:
URL: https://github.com/apache/ozone/pull/11263

   ## What changes were proposed in this pull request?
   
   TestBlockDataStreamOutput.closeAfterWriteFailureThrows checks the injected 
failure message in the exception thrown by close().
   Depending on thread scheduling, the failure is wrapped differently, and the 
outer exception message may not contain the injected message.
   
   The test also checks whether the exception is a CompletionException after 
asserting that it is an IOException, making that branch unreachable. The 
remaining branch contains a no-op assertion; correcting it exposes the 
intermittent failure.
   
   This change verifies the root cause type and message while retaining the 
assertion that close() throws IOException. It removes the unreachable branch 
and updates the comment to explain the asynchronous exception wrapping.
   
   ## What is the link to the Apache JIRA
   
   [HDDS-16482](https://issues.apache.org/jira/browse/HDDS-16482)
   
   ## How was this patch tested?
   
   Repeated closeAfterWriteFailureThrows in IntelliJ IDEA using "N times" set 
to 100 across multiple runs. All runs passed without failures.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to