This is an automated email from the ASF dual-hosted git repository.
agrove 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 1ded76990 Minor: remove code that is now in arrow (#2511)
1ded76990 is described below
commit 1ded76990c616ac1e0a319a870c02dc466b98c8e
Author: Andrew Lamb <[email protected]>
AuthorDate: Wed May 11 10:36:44 2022 -0400
Minor: remove code that is now in arrow (#2511)
---
datafusion/physical-expr/src/expressions/binary.rs | 8 --------
1 file changed, 8 deletions(-)
diff --git a/datafusion/physical-expr/src/expressions/binary.rs
b/datafusion/physical-expr/src/expressions/binary.rs
index 25beea5b2..b7a0da3e0 100644
--- a/datafusion/physical-expr/src/expressions/binary.rs
+++ b/datafusion/physical-expr/src/expressions/binary.rs
@@ -61,14 +61,6 @@ use datafusion_common::{DataFusionError, Result};
use datafusion_expr::binary_rule::binary_operator_data_type;
use datafusion_expr::{binary_rule::coerce_types, ColumnarValue, Operator};
-// TODO move to arrow_rs
-// https://github.com/apache/arrow-rs/issues/1312
-fn as_decimal_array(arr: &dyn Array) -> &DecimalArray {
- arr.as_any()
- .downcast_ref::<DecimalArray>()
- .expect("Unable to downcast to typed array to DecimalArray")
-}
-
/// create a `dyn_op` wrapper function for the specified operation
/// that call the underlying dyn_op arrow kernel if the type is
/// supported, and translates ArrowError to DataFusionError
