François Garillot created ARROW-7692:
----------------------------------------

             Summary: [Rust] Several pattern matches are hard to read
                 Key: ARROW-7692
                 URL: https://issues.apache.org/jira/browse/ARROW-7692
             Project: Apache Arrow
          Issue Type: Improvement
          Components: Rust
            Reporter: François Garillot


Several pattern matches can be rewritten directly using a combinator, e.g. 
array's `value_as_date`, more succintly expressed as a `map`:

{{ match self.value_as_datetime(i) {}}
{{ Some(datetime) => Some(datetime.date()),}}
{{ None => None,}}
{{ }}}



More importantly some of these matches obscure what the code is doing, e.g. 
parquet column writer `read_fully`'s extraction of a mutable slice:
 {{let actual_def_levels = match &mut def_levels {}}
{{ Some(ref mut vec) => Some(&mut vec[..]),}}
{{ None => None,}}
{{ };}}
which can be written, using `as_mut` and `map`, as:

{{let actual_def_levels = def_levels.as_mut().map(|vec| &mut vec[..]);}}

A large # of these are meant to be addressed in 
[https://github.com/apache/arrow/pull/6292/files]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to