tustvold commented on code in PR #2140:
URL: https://github.com/apache/arrow-rs/pull/2140#discussion_r928298669
##########
arrow/src/array/array_decimal.rs:
##########
@@ -368,19 +374,13 @@ impl From<ArrayData> for Decimal256Array {
}
}
-impl<'a> IntoIterator for &'a Decimal128Array {
- type Item = Option<i128>;
- type IntoIter = DecimalIter<'a>;
-
- fn into_iter(self) -> Self::IntoIter {
- DecimalIter::<'a>::new(self)
- }
-}
-
impl<'a> Decimal128Array {
- /// constructs a new iterator
- pub fn iter(&'a self) -> DecimalIter<'a> {
- DecimalIter::new(self)
+ /// Constructs a new iterator that iterates `Decimal128` values as i128
values.
+ /// This is kept mostly for back-compatibility purpose.
+ /// Suggests to use `iter()` that returns `Decimal128Iter`.
+ #[allow(deprecated)]
Review Comment:
Worth adding the above comment as a deprecation hint?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
