Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Nacos Client] There isn't any log output in processServiceInfo in ServiceInfoHolder while empty or error push occurred. #11526

Closed
MajorHe1 opened this issue Dec 19, 2023 · 0 comments · Fixed by #11527
Closed

[Nacos Client] There isn't any log output in processServiceInfo in ServiceInfoHolder while empty or error push occurred. #11526

MajorHe1 opened this issue Dec 19, 2023 · 0 comments · Fixed by #11527
Assignees
@MajorHe1
Labels
area/Client Related to Nacos Client SDK kind/code quality

Comments

@MajorHe1
Copy link
Collaborator

[Nacos Client] There isn't any log output in processServiceInfo in ServiceInfoHolder while empty or error push occurred.

生产实践中，服务端向客户端推送实例变化的消息，客户端收到 NotifySubscriberRequest 并正确回复响应，但是出现了客户端内存中的服务实例信息没有任何变化，导致调用失败的情况。

ServcieInfoHolder类的processServiceInfo 在异常分支里面没有任何日志输出，跟定位问题带来了困难
@MajorHe1 MajorHe1 mentioned this issue Dec 19, 2023
Merged
5 tasks
@MajorHe1 MajorHe1 self-assigned this Dec 19, 2023
@MajorHe1 MajorHe1 added kind/code quality area/Client Related to Nacos Client SDK labels Dec 19, 2023
KomachiSion pushed a commit that referenced this issue Jan 5, 2024
@MajorHe1
[ISSUE #11526] Update ServiceInfoHolder.java, add log output in proce…
dbb1b65 
…ssServiceInfo (#11527)

* Update ServiceInfoHolder.java, add log output in processServiceInfo

* Update ServiceInfoHolder.java, add log output in processServiceInfo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MajorHe1 MajorHe1

Labels
area/Client Related to Nacos Client SDK kind/code quality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #11526] Update ServiceInfoHolder.java, add log output in processServiceInfo MajorHe1/nacos
1 participant
@MajorHe1