jayzhan211 commented on issue #7352:
URL: 
https://github.com/apache/arrow-datafusion/issues/7352#issuecomment-1722224133

   Is it ok to wrap any ArrayRef at the first step?
   
   Not only ListArray(PrimitiveArray(1)) but also PrimitiveArray(1) are wrapped 
in ScalarValue::List.
   
   In the current implementation.
   ScalarValue::Int64(1) can transform to PrimitiveArray(1) and it is 
reversible.
   
   In my current design,
   After ScalarValue::Int64(1) transform to PrimitiveArray(1), it will becomes 
ScalarValue::List and cant transform back to ScalarValue::Int64(1) via 
`try_from_array()`. 
   
   I think it is fine since we do not need to guarantee the ScalarValue and 
Array mapping is one-to-one.


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