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

   ## What changes were proposed in this pull request?
   This error occurs because the `SCMException` is thrown as a 
`RemoteException` to the Client 
(`StorageContainerLocationProtocolClientSideTranslatorPB`) from the server side 
(`StorageContainerLocationProtocolServerSideTranslatorPB`). We unwrap the 
exception using `RemoteException#unwrapRemoteException()`. This function 
expects a constructor with a `String.class` parameter, which is not present in 
`SCMException` and `NonRetriableException`. This results in unwrap failure, and 
the commands are retried until the client exhausts the retry policy.
   
   Currently, `SCMHAUtils#checkNonRetriableException()` is used to fail 
non-retriable commands. It fails to unwrap exceptions, resulting in incorrect 
retry logic. 
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-10760
   
   ## How was this patch tested?
   
   Manual testing with docker.
   


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