alamb opened a new issue, #3042:
URL: https://github.com/apache/arrow-rs/issues/3042

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   While working on https://github.com/apache/arrow-datafusion/pull/4039 and 
https://github.com/influxdata/influxdb_iox/pull/6061 I ended up having to do 
this many times:
   
   ```rust
   Arc::new(TimestampNanosecondArray::from(vec![...],)
     .with_timezone("+0:00".into())
   ```
   
   While this API is already nicer than previously, there are two more 
improvements that might help:
   
   **Describe the solution you'd like**
   Allow passing `&str` as well as `String` to `with_timezone`:
   
   ```rust
   Arc::new(TimestampNanosecondArray::from(vec![...],)
     .with_timezone("+0:00")
   ```
   
   Add an alias for UTC:
   ```rust
   Arc::new(TimestampNanosecondArray::from(vec![...],)
     .with_timezone_utc()
   ```
   
   
   **Describe alternatives you've considered**
   <!--
   A clear and concise description of any alternative solutions or features 
you've considered.
   -->
   
   **Additional context**
   cc @waitingkuo  who is actively working to improve the timezone situation in 
arrow / datafusion ❤️ 
   
   I think this is a good first issue as the code and tests are already there, 
all that is needed is a signature change (from `String` to `impl Into<String>`) 
and some tests


-- 
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]

Reply via email to