Michael-J-Ward opened a new issue, #729:
URL: https://github.com/apache/datafusion-python/issues/729

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   Currently, we de-structure every `ScalarValue` variant
   
   
https://github.com/apache/datafusion-python/blob/860283a466ddefb47c3fce9299f38cd323bc6933/src/expr.rs#L308-L319
   
   But `ScalarValue` already implements `IntoPy`:
   
https://github.com/apache/datafusion/blob/b7d2aea1dd4bb4a3abe3163dae936d7bfa5b32c9/datafusion/common/src/pyarrow.rs#L72-L76
   
   If we want to avoid that `unwrap`, we could use the `ToPyArrow` trait:
   
https://github.com/apache/datafusion/blob/b7d2aea1dd4bb4a3abe3163dae936d7bfa5b32c9/datafusion/common/src/pyarrow.rs#L55-L64
   
   **Additional context**
   Are there other constraints the `ScalarValue` variants that 
`datafusion-python` can support other than what `datafusion` can convert?


-- 
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: github-unsubscr...@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to