chore: sort and dedup formats in sql enginer runner

Signed-off-by: Alexander Droste <alexander.droste@protonmail.com>

Author: 0ax1

vortex-bench/src/lib.rs

@@ -124,7 +124,9 @@ impl Display for Target {
     }
 }
 
-#[derive(Clone, Copy, Debug, Hash, PartialEq, Eq, ValueEnum, Serialize, Deserialize)]
+#[derive(
+    Clone, Copy, Debug, Hash, PartialEq, Eq, PartialOrd, Ord, ValueEnum, Serialize, Deserialize,
+)]
 #[serde(rename_all = "kebab-case")]
 pub enum Format {
     #[clap(name = "csv")]

vortex-bench/src/runner.rs

@@ -3,6 +3,7 @@
 
 //! Generic benchmark runner infrastructure to reduce boilerplate across engine-specific benchmarks.
 
+use std::collections::BTreeSet;
 use std::fs::File;
 use std::future::Future;
 use std::io::Write;
@@ -67,7 +68,8 @@ pub struct SqlBenchmarkRunner {
     benchmark_dataset: BenchmarkDataset,
     storage: String,
     expected_row_counts: Option<Vec<usize>>,
-    formats: Vec<Format>,
+    /// Deduplicated and deterministically ordered.
+    formats: BTreeSet<Format>,
     memory_tracker: Option<BenchmarkMemoryTracker>,
     hide_progress_bar: bool,
     query_measurements: Vec<QueryMeasurement>,
@@ -79,10 +81,11 @@ impl SqlBenchmarkRunner {
     pub fn new<B: Benchmark + ?Sized>(
         benchmark: &B,
         engine: Engine,
-        formats: Vec<Format>,
+        formats: impl IntoIterator<Item = Format>,
         track_memory: bool,
         hide_progress_bar: bool,
     ) -> anyhow::Result<Self> {
+        let formats: BTreeSet<Format> = formats.into_iter().collect();
         let storage = url_scheme_to_storage(benchmark.data_url())?;
 
         let memory_tracker = track_memory.then(BenchmarkMemoryTracker::new);
@@ -101,7 +104,7 @@ impl SqlBenchmarkRunner {
     }
 
     /// Get the formats to run benchmarks for.
-    pub fn formats(&self) -> &[Format] {
+    pub fn formats(&self) -> &BTreeSet<Format> {
         &self.formats
     }
 
@@ -412,7 +415,7 @@ pub fn export_results<W: Write>(
     memory: Vec<MemoryMeasurement>,
     display_format: &DisplayFormat,
     engine: Engine,
-    formats: &[Format],
+    formats: &BTreeSet<Format>,
     mut output: W,
 ) -> anyhow::Result<()> {
     let targets = formats