removed oltp-set run

Signed-off-by: Harshit Gangal <[email protected]>

config/benchmarks/olap-readonly.yaml

@@ -3,7 +3,7 @@ exec-workload: oltp-readonly-olap
 exec-schema: "./vitess-benchmark/sysbench.json"
 
 ## Minimum Vitess version on which the benchmark should be executed
-minimum-version: 14
+minimum-version: 18
 
 ## Ansible
 ansible-inventory-file: macrobench_sharded_inventory.yml
@@ -41,7 +41,5 @@ macrobench_run_warmup-time: 20
 
 # Vitess Extra Flags
 exec-vitess-config:
-  12: # will match >= v12.0.0
-    vtgate: -mysql_default_workload=OLAP
-  14: # will match >= v14.0.0
-    vtgate: --mysql_default_workload=OLAP
+  18: # will match >= v19.0.0
+    vtgate: -mysql_default_workload=OLAP

go/server/server.go

@@ -182,12 +182,11 @@ func (s *Server) Init() error {
 	s.benchmarkConfig = map[string]benchmarkConfig{
 		// "micro":         {file: path.Join(s.benchmarkConfigPath, "micro.yaml"), v: viper.New(), skip: true},
 		"oltp":          {file: path.Join(s.benchmarkConfigPath, "oltp.yaml"), v: viper.New()},
-		"oltp-set":      {file: path.Join(s.benchmarkConfigPath, "oltp-set.yaml"), v: viper.New()},
 		"oltp-readonly": {file: path.Join(s.benchmarkConfigPath, "oltp-readonly.yaml"), v: viper.New()},
 
 		// TODO: oltp-readonly-olap benchmarks are skipped for now as they fail very often due to
 		// MySQL connections being dropped. This issue will be investigated soon.
-		"oltp-readonly-olap": {file: path.Join(s.benchmarkConfigPath, "olap-readonly.yaml"), v: viper.New(), skip: true},
+		// "oltp-readonly-olap": {file: path.Join(s.benchmarkConfigPath, "olap-readonly.yaml"), v: viper.New(), skip: true},
 
 		"tpcc":              {file: path.Join(s.benchmarkConfigPath, "tpcc.yaml"), v: viper.New()},
 		"tpcc_unsharded":    {file: path.Join(s.benchmarkConfigPath, "tpcc_unsharded.yaml"), v: viper.New()},
@@ -199,7 +198,8 @@ func (s *Server) Init() error {
 		if err := config.v.ReadInConfig(); err != nil {
 			slog.Error(err)
 		}
-		if workload == "micro" {
+		// skipping tpcc unsharded and fk workload for comparison
+		if workload == "micro" || strings.Contains(workload, "tpcc_") {
 			continue
 		}
 		s.workloads = append(s.workloads, strings.ToUpper(workload))