Revert "fix: pr benchmark results (#4141)" (#4142)

This reverts commit faf6346. 

The original change messes with the benchmark website.

Signed-off-by: Alexander Droste <[email protected]>

Author: 0ax1

bench-vortex/src/datasets/mod.rs

@@ -39,12 +39,12 @@ pub enum BenchmarkDataset {
 }
 
 impl BenchmarkDataset {
-    pub fn name(&self) -> String {
+    pub fn name(&self) -> &str {
         match self {
-            BenchmarkDataset::TpcH { scale_factor } => format!("tpch_sf{scale_factor}"),
-            BenchmarkDataset::TpcDS { scale_factor } => format!("tpcds_sf{scale_factor}"),
-            BenchmarkDataset::ClickBench { .. } => "clickbench".to_string(),
-            BenchmarkDataset::PublicBi { .. } => "public-bi".to_string(),
+            BenchmarkDataset::TpcH { .. } => "tpch",
+            BenchmarkDataset::TpcDS { .. } => "tpcds",
+            BenchmarkDataset::ClickBench { .. } => "clickbench",
+            BenchmarkDataset::PublicBi { .. } => "public-bi",
         }
     }
 }

scripts/compare-benchmark-jsons.py

@@ -16,10 +16,6 @@
 base = pd.read_json(sys.argv[1], lines=True)
 pr = pd.read_json(sys.argv[2], lines=True)
 
-# Filter duplicate entries which can result from nightly
-# runs if the commit is identical to the base commit of the PR.
-base = base.drop_duplicates(subset=["name", "storage"], keep="last")
-
 base_commit_id = set(base["commit_id"].unique())
 pr_commit_id = set(pr["commit_id"].unique())