From 1675effe8ca814b367704fc6f18683ca1fe721ad Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 17 Jul 2026 10:46:17 +0000 Subject: [PATCH] Rename vortex_bench_dir to bench_results_dir Inside the vortex-bench crate the vortex_bench_ prefix is redundant and stutters at call sites (vortex_bench::output::vortex_bench_dir), and the old name did not say what the directory is for. The new name matches the doc comment: the base directory for benchmark results. Signed-off-by: "Joe Isaacs" Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01XmiEZHiQf7joDWHFF9YSXo --- vortex-bench/src/output.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/vortex-bench/src/output.rs b/vortex-bench/src/output.rs index 7e8a37b7ad7..fe8897f63f9 100644 --- a/vortex-bench/src/output.rs +++ b/vortex-bench/src/output.rs @@ -82,7 +82,9 @@ impl BenchmarkOutput { /// /// Returns `target/vortex-bench//results.json`. pub fn default_output_path(benchmark_id: &str) -> PathBuf { - vortex_bench_dir().join(benchmark_id).join(DEFAULT_FILENAME) + bench_results_dir() + .join(benchmark_id) + .join(DEFAULT_FILENAME) } /// Create an appropriate writer based on display format and output path. @@ -106,10 +108,10 @@ pub fn create_output_writer( } } -/// Get the base directory for vortex benchmark results. +/// Get the base directory for benchmark results. /// /// Returns `target/vortex-bench/`. -pub fn vortex_bench_dir() -> PathBuf { +pub fn bench_results_dir() -> PathBuf { workspace_root().join("target").join("vortex-bench") }