-
Notifications
You must be signed in to change notification settings - Fork 225
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
Upgrade the PD client to adopt the latest refactor #1525
Conversation
667b9ab
to
3e47981
Compare
integration_tests/pd_api_test.go
Outdated
@@ -49,7 +50,7 @@ type apiTestSuite struct { | |||
func (s *apiTestSuite) SetupTest() { | |||
require := s.Require() | |||
addrs := strings.Split(*pdAddrs, ",") | |||
pdClient, err := pd.NewClient(addrs, pd.SecurityOption{}) | |||
pdClient, err := pd.NewClient(caller.Component("client-go-integration-test"), addrs, pd.SecurityOption{}) |
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.
@okJiang Do we have a standard for the definition of the component?
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.
If it is in the test, we can use caller.TestComponent. Otherwise, we can put it empty now, I will update them in #1516
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.
BTW, how about adding a DefaultComponent = ""
in pd repo?
Signed-off-by: JmPotato <[email protected]>
Signed-off-by: JmPotato <[email protected]>
Signed-off-by: JmPotato <[email protected]>
588f013
to
1b9895b
Compare
Signed-off-by: JmPotato <[email protected]>
1b9895b
to
1dcf3e6
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rleungx, you06 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Integration testing can only resume normal operation after TiDB has completed its update.