From 242f5abe2a59f617b14ba98db85b5aecbc51de81 Mon Sep 17 00:00:00 2001 From: Siew Kam Onn Date: Fri, 3 Jan 2025 16:34:38 +0800 Subject: [PATCH] Rename spill pool test functions for clarity and consistency --- datafusion/core/src/dataframe/mod.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/datafusion/core/src/dataframe/mod.rs b/datafusion/core/src/dataframe/mod.rs index d31ce5d06bf7..2085e05e2d65 100644 --- a/datafusion/core/src/dataframe/mod.rs +++ b/datafusion/core/src/dataframe/mod.rs @@ -2755,7 +2755,7 @@ mod tests { } // test for https://github.com/apache/datafusion/issues/13949 - async fn run_test_with_spill_pool(pool_size: usize) -> Result<()> { + async fn run_test_with_spill_pool_if_necessary(pool_size: usize) -> Result<()> { fn create_record_batch( schema: &Arc, data: (Vec, Vec), @@ -2844,11 +2844,11 @@ mod tests { } #[tokio::test] - async fn test_aggregate_with_spill() -> Result<()> { + async fn test_aggregate_with_spill_if_necessary() -> Result<()> { // test with spill - run_test_with_spill_pool(1600).await?; + run_test_with_spill_pool_if_necessary(1600).await?; // test without spill - run_test_with_spill_pool(16000).await?; + run_test_with_spill_pool_if_necessary(16000).await?; Ok(()) }