Added support for passing a tag with the test

test/cpp/qps/perf_db.proto

@@ -62,9 +62,10 @@ message DataDetails {
   string timestamp = 1;
   string test_name = 2;
   string sys_info = 3;
-  Metrics metrics = 4;
-  ClientConfig client_config = 5;
-  ServerConfig server_config = 6;
+  string tag = 4;
+  Metrics metrics = 5;
+  ClientConfig client_config = 6;
+  ServerConfig server_config = 7;
 }
 
 //User details
@@ -93,9 +94,10 @@ message SingleUserRecordRequest {
   string access_token = 1;
   string test_name = 2;
   string sys_info = 3;
-  Metrics metrics = 4;
-  ClientConfig client_config = 5;
-  ServerConfig server_config = 6;
+  string tag = 4;
+  Metrics metrics = 5;
+  ClientConfig client_config = 6;
+  ServerConfig server_config = 7;
 }
 
 //Reply to request for storing single user's data

test/cpp/qps/perf_db_client.cc

@@ -72,7 +72,7 @@ void PerfDbClient::setTimes(double serverSystemTime, double serverUserTime,
 }
 
 //sends the data to the performancew database server
-int PerfDbClient::sendData(std::string access_token, std::string test_name, std::string sys_info) {
+int PerfDbClient::sendData(std::string access_token, std::string test_name, std::string sys_info, std::string tag) {
   //Data record request object
   SingleUserRecordRequest singleUserRecordRequest;
 
@@ -80,6 +80,7 @@ int PerfDbClient::sendData(std::string access_token, std::string test_name, std:
   singleUserRecordRequest.set_access_token(access_token);
   singleUserRecordRequest.set_test_name(test_name);
   singleUserRecordRequest.set_sys_info(sys_info);
+  singleUserRecordRequest.set_tag(tag);
 
   //setting configs
   *(singleUserRecordRequest.mutable_client_config()) = this->clientConfig_;

test/cpp/qps/perf_db_client.h

@@ -88,7 +88,7 @@ public:
     double clientSystemTime, double clientUserTime);
 
   //sends the data to the performancew database server
-  int sendData(std::string access_token, std::string test_name, std::string sys_info);
+  int sendData(std::string access_token, std::string test_name, std::string sys_info, std::string tag);
 
 private:
   std::unique_ptr<PerfDbTransfer::Stub> stub_;

test/cpp/qps/report.cc

@@ -173,7 +173,7 @@ void PerfDbReporter::ReportTimes(const ScenarioResult& result) {
 
 void PerfDbReporter::SendData() {
   //send data to performance database
-  int dataState = perfDbClient_.sendData(access_token_, test_name_, sys_info_);
+  int dataState = perfDbClient_.sendData(access_token_, test_name_, sys_info_, tag_);
 
   //check state of data sending
   switch(dataState) {

test/cpp/qps/report.h

@@ -107,8 +107,9 @@ class GprLogReporter : public Reporter {
 /** Reporter for performance database tool */
 class PerfDbReporter : public Reporter {
  public:
-  PerfDbReporter(const string& name, const string& access_token, const string& test_name, const string& sys_info, const string& server_address)
-   : Reporter(name), access_token_(access_token), test_name_(test_name), sys_info_(sys_info) {
+  PerfDbReporter(const string& name, const string& access_token, const string& test_name, 
+    const string& sys_info, const string& server_address, const string& tag)
+   : Reporter(name), access_token_(access_token), test_name_(test_name), sys_info_(sys_info), tag_(tag) {
     perfDbClient_.init(grpc::CreateChannel(server_address, grpc::InsecureCredentials(), ChannelArguments()));
   }
   ~PerfDbReporter() { SendData(); };
@@ -118,6 +119,7 @@ class PerfDbReporter : public Reporter {
   std::string access_token_;
   std::string test_name_;
   std::string sys_info_;
+  std::string tag_;
   void ReportQPS(const ScenarioResult& result) GRPC_OVERRIDE;
   void ReportQPSPerCore(const ScenarioResult& result) GRPC_OVERRIDE;
   void ReportLatency(const ScenarioResult& result) GRPC_OVERRIDE;

test/cpp/util/benchmark_config.cc

@@ -47,6 +47,8 @@ DEFINE_string(sys_info, "", "System information");
 
 DEFINE_string(server_address, "localhost:50052", "Address of the performance database server");
 
+DEFINE_string(tag, "", "Optional tag for the test");
+
 // In some distros, gflags is in the namespace google, and in some others,
 // in gflags. This hack is enabling us to find both.
 namespace google {}
@@ -69,7 +71,8 @@ static std::shared_ptr<Reporter> InitBenchmarkReporters() {
   }
   if(FLAGS_report_metrics_db) {
     composite_reporter->add(
-      std::unique_ptr<Reporter>(new PerfDbReporter("PerfDbReporter", FLAGS_access_token, FLAGS_test_name, FLAGS_sys_info, FLAGS_server_address)));
+      std::unique_ptr<Reporter>(new PerfDbReporter("PerfDbReporter", FLAGS_access_token, FLAGS_test_name, 
+        FLAGS_sys_info, FLAGS_server_address, FLAGS_tag)));
   }
 
   return std::shared_ptr<Reporter>(composite_reporter);