(sedona-db) branch jw/gdal-loader-concurrency created (now 5d21f81a4)

[ja...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

james-willis pushed a change to branch jw/gdal-loader-concurrency
in repository https://gitbox.apache.org/repos/asf/sedona-db.git


      at 5d21f81a4 test(rust/sedona-raster-gdal): number bench generations across both sweeps

This branch includes the following new commits:

     new 5d21f81a4 test(rust/sedona-raster-gdal): number bench generations across both sweeps

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

(sedona-db) 01/01: test(rust/sedona-raster-gdal): number bench generations across both sweeps

[ja...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

james-willis pushed a commit to branch jw/gdal-loader-concurrency
in repository https://gitbox.apache.org/repos/asf/sedona-db.git

commit 5d21f81a4ab262fa75e4f4db8659a7abff6415e2
Author: jameswillis <ja...@wherobots.com>
AuthorDate: Fri Oct 2 10:35:12 2026 -0700

    test(rust/sedona-raster-gdal): number bench generations across both sweeps
---
 rust/sedona-raster-gdal/benches/gdal_io_concurrency.rs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/rust/sedona-raster-gdal/benches/gdal_io_concurrency.rs b/rust/sedona-raster-gdal/benches/gdal_io_concurrency.rs
index 7635e4a3e..1a8f59c98 100644
--- a/rust/sedona-raster-gdal/benches/gdal_io_concurrency.rs
+++ b/rust/sedona-raster-gdal/benches/gdal_io_concurrency.rs
@@ -141,8 +141,8 @@ fn sweep(
     root: &Path,
     prefix: &str,
     rt: &tokio::runtime::Runtime,
+    generation: &AtomicUsize,
 ) {
-    let generation = AtomicUsize::new(0);
     let mut group = c.benchmark_group(group_name);
     group.sample_size(10);
     for callers in CALLERS {
@@ -259,6 +259,9 @@ fn bench_io_budget(c: &mut Criterion) {
         .enable_all()
         .build()
         .unwrap();
+    // One counter for both sweeps: they share `tmp`, and each leaves its
+    // last generation's directory behind.
+    let generation = AtomicUsize::new(0);
 
     sweep(
         c,
@@ -266,6 +269,7 @@ fn bench_io_budget(c: &mut Criterion) {
         tmp.path(),
         &tmp.path().display().to_string(),
         &rt,
+        &generation,
     );
 
     let latency_ms: u64 = std::env::var("GDAL_BENCH_LATENCY_MS")
@@ -281,6 +285,7 @@ fn bench_io_budget(c: &mut Criterion) {
                 tmp.path(),
                 &prefix,
                 &rt,
+                &generation,
             );
         }
         None => eprintln!("python3 not available or server did not start; skipping the HTTP store"),