[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
代码bug比较多: * 超时时间完全不生效,不管在哪一端设置; * 服务端抛异常,客户端接不到。 * RpcContext不生效 [ Full content available at: https://github.com/apache/dubbo/issues/3999 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
有几个点我觉得有风险: * 性能表现,我看到代码里有直接走反射的,没有细看,不确定这个反射是否只执行1次; * 资源回收,我看 [motan的GRPC实现](https://github.com/weibocom/motan/blob/master/motan-extension/protocol-extension/motan-protocol-grpc/src/main/java/com/weibo/api/motan/protocol/grpc/GrpcProtocol.java) 有很多资源回收的处置; * 插件维护,protobuf编译插件是直接在C代码上改的,javae

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
另外,测过性能没有?我看核心链路上直接走反射,比如: ```java /** * 这里返回的impl必须要有所有的方法, Stub BlockingStube FutureStub */ @Override protected T doRefer(Class type, URL url) throws RpcException { //通过 type(IGreeter),反射获得外部类(GreeterGrpc),调用静态方法(getDubboStub()),获得并返回(T)stub Class enclosing

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
另外,测过性能没有?我看核心链路上直接走反射,比如: ```java /** * 这里返回的impl必须要有所有的方法, Stub BlockingStube FutureStub */ @Override protected T doRefer(Class type, URL url) throws RpcException { //通过 type(IGreeter),反射获得外部类(GreeterGrpc),调用静态方法(getDubboStub()),获得并返回(T)stub Class enclosing

[GitHub] [dubbo] konglz commented on issue #3999: [SoC2019]Supporting gRPC Protocol.

2019-09-10 Thread GitHub
代码bug比较多: * 1. 超时时间完全不生效,不管在哪一端设置; * 2. 服务端抛异常,客户端接不到。 [ Full content available at: https://github.com/apache/dubbo/issues/3999 ] This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org - To unsubscrib