Skip to content

Commit

Permalink
Add bi-directional gRPC streaming options for periodic concurrency mode
Browse files Browse the repository at this point in the history
  • Loading branch information
nv-hwoo committed Sep 6, 2023
1 parent a9cc84b commit ee7db60
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions src/c++/perf_analyzer/test_command_line_parser.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1194,8 +1194,13 @@ TEST_CASE("Testing Command Line Parser")
char* option_name = "--periodic-concurrency-range";

// Add required args that specifies where to dump profiled data
args.push_back("--profile-export-file");
args.push_back("profile.json");
args.insert(
args.end(), {"-i", "grpc", "--async", "--streaming",
"--profile-export-file", "profile.json"});
exp->protocol = cb::ProtocolType::GRPC;
exp->async = true;
exp->streaming = true;
exp->url = "localhost:8001"; // gRPC url

CheckValidRange(
args, option_name, parser, act, exp->using_periodic_concurrency_range,
Expand Down Expand Up @@ -1229,7 +1234,7 @@ TEST_CASE("Testing Command Line Parser")

SUBCASE("no export file specified")
{
// Remove the required args
// Remove the export file args
args.pop_back();
args.pop_back();

Expand Down

0 comments on commit ee7db60

Please sign in to comment.