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

NotificationControllerV2的 pollNotification 与handleMessage方法是否有并发问题 #2111

Closed
bkblack opened this issue Apr 3, 2019 · 1 comment · Fixed by #2255
Closed

NotificationControllerV2的 pollNotification 与handleMessage方法是否有并发问题 #2111

bkblack opened this issue Apr 3, 2019 · 1 comment · Fixed by #2255

Comments

@bkblack
Copy link
Contributor

bkblack commented Apr 3, 2019

image

  • 上图1-2之间，如果handleMessage处理以下逻辑，会让上面这个longpolling多等一分钟吧。
    image
@bkblack
Copy link
Contributor Author

bkblack commented Apr 3, 2019

建议颠倒下add deferredResult与fast check的顺序

This was referenced May 13, 2019
Closed
Merged
nobodyiam pushed a commit that referenced this issue May 21, 2019
@bkblack @nobodyiam
fix #2111 (#2255)
469c1c1
ForeverRunner pushed a commit to ForeverRunner/apollo-all that referenced this issue May 28, 2022
@bkblack @xuexiangdong
fix apolloconfig/apollo#2111 (#2255)
1dcccc8
CrackerCat pushed a commit to CrackerCat/apollo-1 that referenced this issue Jul 31, 2024
@bkblack
fix apolloconfig#2111 (apolloconfig#2255)
376e88e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix https://github.com/ctripcorp/apollo/issues/2111
1 participant
@bkblack