When blocked in Dubbo adapter, Sentinel should wrap SentinelRpcException with Result rather than directly throw it out

[lesline]

when block exception happen, sentinel return RpcException exception，not SentinelRpcException exception

Type: bug report

Describe problem:

problem code

com.alibaba.csp.sentinel.adapter.dubbo.fallback.DefaultDubboFallback
public class DefaultDubboFallback implements DubboFallback {
    @Override
    public Result handle(Invoker<?> invoker, Invocation invocation, BlockException ex) {
        // Just wrap and throw the exception.
        throw new SentinelRpcException(ex);
    }
}

should change to :

public class DefaultDubboFallback implements DubboFallback {

    @Override
    public Result handle(Invoker<?> invoker, Invocation invocation, BlockException ex) {
        // Just wrap and throw the exception.
        RpcResult result = new RpcResult();
        result.setException(new SentinelRpcException(ex));
        return result;
    }
}

Tell us your environment

<version>1.7.2-SNAPSHOT</version>

[sczyh30]

Good idea. Would you like to contribute a PR to improve it?

[lesline]

ok

[sczyh30]

PRs are welcomed for Apache Dubbo 2.7.x adapter!

[cdfive]

Maybe relate to #374, #1065, #1017, when using DefaultDubboFallback, we need to be careful about Dubbo ExceptionFilter, Serialize/Deserialize and JDK vension.

[wzg923]

#1065

[sczyh30]

Already resolved in 1.8.x.