waibozie commented on issue #2571: URL: https://github.com/apache/servicecomb-java-chassis/issues/2571#issuecomment-917795308
> > > 方便提交一个PR修改下吗？ > > > > > > 我看最新分支上已经把`servicecomb.service.registry.client.timeout.watch`配置项删除了，确认一下这是有意为之还是误操作？ > > 修改的提交记录：[cb-2317](https://github.com/apache/servicecomb-java-chassis/pull/2531/files)， contributor是@develpoerX > > 讨论确认一下。 > > 你说的这个问题，是一个无效的方法，在删除之前，它在整个项目中没有任何地方被使用到，所以我就删除了它。我猜测之前应该是有操作吧这个配置项删除，或者有存在预设了一个配置项，但是还没有想好使用场景，因此没有用到这个配置。 从git提交记录看，这个配置是一个非常老的提交：[[JAV-442]add PULL support between SDK and service center](https://github.com/apache/servicecomb-java-chassis/commit/1626b64a1bbf262420096214577317fe7f784cd5) ```java // WebSocketClientPool tag 1.2.0 @Override public HttpClientOptions createHttpClientOptions() { // ignore httpClientOptions.setIdleTimeout(ServiceRegistryConfig.INSTANCE.getIdleWatchTimeout()); // ignore } ``` 我记得注册中心的websocket策略的30s一次，且不接受客户端的ping，只允许注册中心主动ping客户端， 在我的场景里我把这个值调整得比心跳的超时的链路超时长得多。 2.X的重构确实破坏了这个配置项，我个人觉得把它独立出来还是有必要的， 比较注册中心的ping/pong策略比较特别。 @liubao68 @develpoerX 你们觉得呢？ -- 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]
