You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by dh...@apache.org on 2021/05/23 08:39:24 UTC

[arrow-datafusion] branch master updated: Add PartialOrd and Ord to GroupByScalar (#364) (#368)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 76ac914  Add PartialOrd and Ord to GroupByScalar (#364) (#368)
76ac914 is described below

commit 76ac91440e759c59112b70450e970cc99776ec5e
Author: Raphael Taylor-Davies <17...@users.noreply.github.com>
AuthorDate: Sun May 23 09:39:17 2021 +0100

    Add PartialOrd and Ord to GroupByScalar (#364) (#368)
    
    Signed-off-by: Raphael Taylor-Davies <r....@googlemail.com>
---
 datafusion/src/physical_plan/group_scalar.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datafusion/src/physical_plan/group_scalar.rs b/datafusion/src/physical_plan/group_scalar.rs
index 943386d..d5f72b0 100644
--- a/datafusion/src/physical_plan/group_scalar.rs
+++ b/datafusion/src/physical_plan/group_scalar.rs
@@ -24,7 +24,7 @@ use crate::error::{DataFusionError, Result};
 use crate::scalar::ScalarValue;
 
 /// Enumeration of types that can be used in a GROUP BY expression
-#[derive(Debug, PartialEq, Eq, Hash, Clone)]
+#[derive(Debug, PartialEq, Eq, PartialOrd, Ord, Hash, Clone)]
 pub(crate) enum GroupByScalar {
     Float32(OrderedFloat<f32>),
     Float64(OrderedFloat<f64>),