-
Notifications
You must be signed in to change notification settings - Fork 148
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
Feat: Pixiu connect with SpringCloud Server by nacos #255
Conversation
merge origin develop
'get nacos client'
…o-go-pixiu into PhilYue-feature/springcloud
The rest code of get nacos client
Codecov Report
@@ Coverage Diff @@
## develop #255 +/- ##
===========================================
- Coverage 39.75% 39.56% -0.20%
===========================================
Files 50 50
Lines 2568 2626 +58
===========================================
+ Hits 1021 1039 +18
- Misses 1436 1475 +39
- Partials 111 112 +1
Continue to review full report at Codecov.
|
Feature/springcloud
import regroup
pkg/registry/zookeeper.go
Outdated
@@ -18,6 +18,7 @@ | |||
package registry | |||
|
|||
import ( | |||
"github.com/apache/dubbo-go-pixiu/pkg/model" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move it to the 3rd import block.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
} | ||
|
||
ServiceDiscovery interface { | ||
// 直接向远程注册中心查询所有服务实例 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
translate it into english
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
pkg/adapter/springcloud/cloud.go
Outdated
a.mutex.Lock() | ||
defer a.mutex.Unlock() | ||
|
||
var res []string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
res := make([]string, 0, len(a.currentService))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Feat: Pixiu connect with SpringCloud Server by nacos Former-commit-id: 32a7ddf
Feat: Pixiu connect with SpringCloud Server by nacos
What this PR does:
Pixiu connect with SpringCloud Server
Which issue(s) this PR fixes:
Fixes #
nope
Special notes for your reviewer:
nope
Does this PR introduce a user-facing change?:
nope