Apply suggestions from code review

Co-authored-by: Yongting You <2010youy01@gmail.com>
Co-authored-by: Dewey Dunnington <dewey@dunnington.ca>

Author: 3 people

rust/sedona/src/memory_pool.rs

@@ -26,7 +26,9 @@ pub const DEFAULT_UNSPILLABLE_RESERVE_RATIO: f64 = 0.2;
 /// A [`MemoryPool`] implementation similar to DataFusion's [`datafusion::execution::memory_pool::FairSpillPool`],
 /// but with the following changes:
 ///
-/// It implements a reservation mechanism for unspillable memory consumers. This addresses an issue
+/// Spillable and non-spillable operators use logically separate portions of the memory pool, 
+/// controlled by unspillable_reserve_ratio, instead of sharing a single pool as in
+/// DataFusion’s default FairSpillPool, which can  lead to the following issues:
 /// where spillable consumers could potentially exhaust all available memory, preventing unspillable
 /// operations from acquiring necessary resources. This behavior is tracked in DataFusion issue
 /// https://github.com/apache/datafusion/issues/17334. In the context of Sedona, a typical example

sedona-cli/src/main.rs

@@ -275,7 +275,7 @@ fn parse_size_string(size: &str, label: &str) -> Result<usize, String> {
     });
 
     static SUFFIX_REGEX: LazyLock<regex::Regex> =
-        LazyLock::new(|| regex::Regex::new(r"^([0-9]+)([a-z]+)?$").unwrap());
+        LazyLock::new(|| regex::Regex::new(r"^([0-9.]+)\s*([a-z]+)?$").unwrap());
 
     let lower = size.to_lowercase();
     if let Some(caps) = SUFFIX_REGEX.captures(&lower) {