-
Notifications
You must be signed in to change notification settings - Fork 4.5k
cleanup: replace dial with newclient #7975
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
Merged
Merged
Changes from 3 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
7d2a53b
replace dial with newclient
janardhankrishna-sai 66a6a32
passing passthrough to the newclient and updating err msg
janardhankrishna-sai 614299d
adding close() for goroutine leak and using fatal instead of log
janardhankrishna-sai beb8b0f
waiting for the test balancer to be built before getting the state no…
janardhankrishna-sai b405e43
replace dial with newclient and updating error msg
janardhankrishna-sai 1748c69
reverting config whitespaces
janardhankrishna-sai be8a69f
reverting config whitespaces
janardhankrishna-sai fe19ecf
updating error msg
janardhankrishna-sai 77953ec
updating error msg
janardhankrishna-sai 3e1fc48
addressing review comments
janardhankrishna-sai 09b067c
using r.InitialState instead of r.UpdateState
janardhankrishna-sai 1fddf6b
defining load balancing grpclb service config
janardhankrishna-sai 6f1e7c2
using load balancing service config
janardhankrishna-sai ab5f146
using internal.ParseServiceConfig to avoid depending on manual resolv…
janardhankrishna-sai 3b75fe7
using const for timeout and lbpolicy, updating comments and removing …
janardhankrishna-sai 14d0d7d
using const for grpclbpolicy
janardhankrishna-sai 1835cfe
adding ctx in callandverify and removing explicit cc.Connect() call
janardhankrishna-sai c163e9f
removing check for len of resolver state address
janardhankrishna-sai d74e2de
updating resolver state and parseserviceconfig
janardhankrishna-sai f377c38
addressing review comments
janardhankrishna-sai ce24eb5
updating comment
janardhankrishna-sai 54a2a01
removing duplicate comments
janardhankrishna-sai 540911c
addressing review comments
janardhankrishna-sai 59a45c7
using parseServiceConfig instead of internal.ParseServiceConfig
janardhankrishna-sai 358cb39
using parseServiceConfig, explicitly calling cc.Connect() and updatin…
janardhankrishna-sai ba8385d
using parseServiceConfig, explicitly calling cc.Connect() and updatin…
janardhankrishna-sai 42ada72
removing comment
janardhankrishna-sai 9834f60
addressing review comments
janardhankrishna-sai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.