You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by vi...@apache.org on 2023/04/12 20:05:54 UTC

[arrow-datafusion] branch main updated: Minor: fix wrong code comment (#5979)

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

viirya pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 826001d601 Minor: fix wrong code comment (#5979)
826001d601 is described below

commit 826001d6019662b34afc866ad389ce1cb6076bf9
Author: Liang-Chi Hsieh <vi...@gmail.com>
AuthorDate: Wed Apr 12 13:05:49 2023 -0700

    Minor: fix wrong code comment (#5979)
---
 datafusion/core/tests/memory_limit.rs | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/datafusion/core/tests/memory_limit.rs b/datafusion/core/tests/memory_limit.rs
index 034505f52c..13b6291f9c 100644
--- a/datafusion/core/tests/memory_limit.rs
+++ b/datafusion/core/tests/memory_limit.rs
@@ -161,12 +161,10 @@ async fn merge_join() {
     .await
 }
 
-/// 50 byte memory limit
 const MEMORY_FRACTION: f64 = 0.95;
 
-/// runs the specified query against 1000 rows with a 50
-/// byte memory limit and no disk manager enabled
-/// with default SessionConfig.
+/// runs the specified query against 1000 rows with specified
+/// memory limit and no disk manager enabled with default SessionConfig.
 async fn run_limit_test(
     query: &str,
     expected_error_contains: Vec<&str>,
@@ -195,7 +193,6 @@ async fn run_limit_test_with_config(
     let rt_config = RuntimeConfig::new()
         // do not allow spilling
         .with_disk_manager(DiskManagerConfig::Disabled)
-        // Only allow 50 bytes
         .with_memory_limit(memory_limit, MEMORY_FRACTION);
 
     let runtime = RuntimeEnv::new(rt_config).unwrap();