nacos icon indicating copy to clipboard operation
nacos copied to clipboard

RpcClient新增系统参数类

Open stone-98 opened this issue 1 year ago • 0 comments

Is your feature request related to a problem? Please describe. RpcClient新增系统参数类,从系统属性中获取与Rpc相关处理分离,在之前的基础上新增:

  • nacos.remote.grpc.executor.keep.alive.millis
  • nacos.remote.grpc.request.timeout.millis
  • nacos.remote.rpc.request.timeout.millis
  • nacos.remote.rpc.request.retry.times
  • nacos.remote.rpc.server.keep.alive.millis

Describe the solution you'd like A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

stone-98 avatar Aug 10 '22 15:08 stone-98

我之前还想该这种类似的东西呢,可是没有人反馈,没想到被你抢先了, 关联一下我之前发的ISSUE吧:#8765

wuchubuzai2018 avatar Aug 11 '22 02:08 wuchubuzai2018

我提一下我的想法:

  1. 确实需要可配置的内容;
  2. 我觉得可以复用社区的另一功能 https://github.com/alibaba/nacos/issues/8622.
  3. 还需要整理一下之前已经提供出去的一些参数,一起统一下

可以整合到一个issue里,把方案都贴在那个issue里,然后讨论出最终结果后再动手写PR。

KomachiSion avatar Aug 11 '22 06:08 KomachiSion

我之前还想该这种类似的东西呢,可是没有人反馈,没想到被你抢先了, 关联一下我之前发的ISSUE吧:#8765

我在群里和issue上看到蛮多相关参数需要这种动态配置。所以就提了。不过我这个可能不太适合。

stone-98 avatar Aug 11 '22 06:08 stone-98

我提一下我的想法:

  1. 确实需要可配置的内容;
  2. 我觉得可以复用社区的另一功能 [Enhance] getting arguments use an unified way in the client module #8622.
  3. 还需要整理一下之前已经提供出去的一些参数,一起统一下

可以整合到一个issue里,把方案都贴在那个issue里,然后讨论出最终结果后再动手写PR。

感谢回复,稍后我会将Nacos-Client端所有已经支持的参数以及需要新增的参数梳理一下。然后重新开一个issue。

stone-98 avatar Aug 12 '22 12:08 stone-98