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

tustvold pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/master by this push:
     new 0b977c6954f Fix test_string_dictionary_merge (#5255) (#5461)
0b977c6954f is described below

commit 0b977c6954ff8de776598ab75f5cf56e72f0d727
Author: Raphael Taylor-Davies <[email protected]>
AuthorDate: Mon Mar 4 09:47:21 2024 +1300

    Fix test_string_dictionary_merge (#5255) (#5461)
    
    * Fix test_string_dictionary_merge (#5255)
    
    * Clippy
---
 arrow-select/src/concat.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arrow-select/src/concat.rs b/arrow-select/src/concat.rs
index 04e3ab2f742..695903195d2 100644
--- a/arrow-select/src/concat.rs
+++ b/arrow-select/src/concat.rs
@@ -569,7 +569,8 @@ mod tests {
 
         // Should have merged inputs together
         // Not 30 as this is done on a best-effort basis
-        assert_eq!(dictionary.values().len(), 33)
+        let values_len = dictionary.values().len();
+        assert!((30..40).contains(&values_len), "{values_len}")
     }
 
     #[test]

Reply via email to