-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
QPS test that uses inproc transport #12362
Conversation
|
|
|
|
|
|
|
|
1 similar comment
|
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.
LGTM. Just some minor suggestions.
test/cpp/qps/server_async.cc
Outdated
// Negative port number means inproc server, so no listen port needed | ||
if (port_num >= 0) { | ||
char *server_address = NULL; | ||
gpr_join_host_port(&server_address, "::", port()); |
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.
Just use port_num?
gpr_join_host_port(&server_address, "::", port_num);
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.
Resolved
test/cpp/qps/server_sync.cc
Outdated
// Negative port number means inproc server, so no listen port needed | ||
if (port_num >= 0) { | ||
char* server_address = NULL; | ||
gpr_join_host_port(&server_address, "::", port()); |
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.
Just use port_num?
gpr_join_host_port(&server_address, "::", port_num);
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.
Resolved
|
|
I will not merge this until #12433 gets resolved |
Rebased master and mixed in #12797 hoping that that makes tests pass that were previously OOMing |
|
0b0ab2f
to
d5da6db
Compare
Rebased and squashed now that #12797 is in. Diff unchanged. |
|
|
|
|
|
|
|
1 similar comment
|
|
1 similar comment
|
19c0b27
to
d21ef70
Compare
|
1 similar comment
|
|
|
|
1 similar comment
|
…to standard testing suite.
|
|
|
|
|
One of the goals of this is to help debug remaining issues with QPS testing