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

[arrow-rs] branch master updated: minor: remove useless prefix (#4367)

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 077058d55 minor: remove useless prefix (#4367)
077058d55 is described below

commit 077058d55c1e27e30f9affdc7c8ac9eef651eccc
Author: jakevin <ja...@gmail.com>
AuthorDate: Tue Jun 6 14:31:14 2023 +0800

    minor: remove useless prefix (#4367)
---
 arrow-ord/src/comparison.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/arrow-ord/src/comparison.rs b/arrow-ord/src/comparison.rs
index b9274f0ea..4f8b9a322 100644
--- a/arrow-ord/src/comparison.rs
+++ b/arrow-ord/src/comparison.rs
@@ -1196,9 +1196,10 @@ where
     K: ArrowDictionaryKeyType,
     K::Native: num::ToPrimitive,
 {
-    // TODO: Use take_boolean (#2967)
-    let array = take(&dict_comparison, dict.keys(), None)?;
-    Ok(BooleanArray::from(array.to_data()))
+    let array = take(&dict_comparison, dict.keys(), None)?
+        .as_boolean()
+        .clone();
+    Ok(array)
 }
 
 /// Helper function to perform boolean lambda function on values from two arrays using