[GitHub] [dubbo] chickenlj commented on pull request #11088: support exception process when service not found

["chickenlj (via GitHub)" <gi...@apache.org>]

chickenlj commented on PR #11088:
URL: https://github.com/apache/dubbo/pull/11088#issuecomment-1467450202

   I didn't look into the details of the previous discussion. By just reviewing the code, I have the following concerns:
   
   1. One more scattered SPI definition and its configuration item,  which in this case is `String exPs = System.getProperty(EXCEPTION_PROCESSOR_KEY);`.
   2. The only functionality of `ExceptionFilter` is to provide users a way to customize `DecodeableRpcInvocation`, would it be better to include the current initialization of `DecodeableRpcInvocation` as a default implementation of `ExceptionFilter`? And in that way, we might want to rename `ExceptionFilter` as something like `DecodeableInvocationProvider`, so, for users who care about this SPI there will have:
     * A SPI definition `DecodeableInvocationProvider`
     * The default SPI implementation `DefaultDecodeableInvocationProvider`
     * The customized SPI implementation `ExceptionDecodeableInvocationProvider`
   
   Besides the implementation, I care more about the use cases and the necessity of introducing this extension.


-- 
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: notifications-unsubscribe@dubbo.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org