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.

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

fix polaris Subscriber #2026

Merged
merged 24 commits into from
Aug 25, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
08af2cc
fix(proto): fix getting attributes issue (#1968)
justxuewei Jul 12, 2022
7017726
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
6747f90
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
20a56d5
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
43cd167
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
c854fba
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
83174a2
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 17, 2022
a8513bd
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 18, 2022
5d7c857
Merge branch 'apache:3.0' into 3.0
jasondeng1997 Aug 18, 2022
511d8e8
Merge branch 'apache:3.0' into 3.0
jasondeng1997 Aug 18, 2022
9543bd9
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 18, 2022
268b46e
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 18, 2022
22aa477
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 20, 2022
833da65
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 20, 2022
b281847
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
54692c3
Merge branch 'apache:3.0' into 3.0
jasondeng1997 Aug 22, 2022
888fee9
Merge branch 'apache:3.0' into 3.0
jasondeng1997 Aug 22, 2022
7fa2a40
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
e304b72
Merge branch 'apache:3.0' into 3.0
jasondeng1997 Aug 22, 2022
3a3335f
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
472d6ef
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
84cf509
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
f6615cc
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
f5a933d
Merge remote-tracking branch 'origin/3.0' into 3.0
jasondeng1997 Aug 22, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'apache:3.0' into 3.0
  • Loading branch information
jasondeng1997 committed Aug 18, 2022
commit 511d8e8f38c12e2e2c84e6aa371ef2db018ee4fd
4 changes: 4 additions & 0 deletions registry/polaris/core_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@ import (
)

func TestPolarisServiceWatcher_AddSubscriber(t *testing.T) {
var (
newParam api.WatchServiceRequest
newConsumer api.ConsumerAPI
)
type fields struct {
consumer api.ConsumerAPI
subscribeParam *api.WatchServiceRequest
Expand Down
7 changes: 5 additions & 2 deletions registry/polaris/registry.go
Original file line number Diff line number Diff line change
Expand Up @@ -150,8 +150,11 @@ func (pr *polarisRegistry) UnRegister(conf *common.URL) error {

// Subscribe returns nil if subscribing registry successfully. If not returns an error.
func (pr *polarisRegistry) Subscribe(url *common.URL, notifyListener registry.NotifyListener) error {
var newParam api.WatchServiceRequest
var newConsumer api.ConsumerAPI
var (
newParam api.WatchServiceRequest
newConsumer api.ConsumerAPI
)

role, _ := strconv.Atoi(url.GetParam(constant.RegistryRoleKey, ""))
if role != common.CONSUMER {
return nil
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.