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();