[GitHub] [arrow-datafusion] alamb commented on pull request #3534: Add serialization of `ScalarValue::Binary` and `ScalarValue::LargeBinary`, `ScalarValue::Time64`

Wed, 21 Sep 2022 09:35:05 -0700 


alamb commented on PR #3534:
URL: 
https://github.com/apache/arrow-datafusion/pull/3534#issuecomment-1253953655

   > that the whole situation is very confusing however. One day I hope someone 
has enough high-level knowledge to clean it up in a sensible way.
   
   I have plans (see https://github.com/apache/arrow-datafusion/pull/3547 ) but 
it has somewhat turned into 
   
   
![yak_hair_overflow](https://user-images.githubusercontent.com/490673/191560891-3cca98d2-493c-4473-a463-e0a08b0b06c4.png)
   
   
   I think I can get remove the entire `PrimitiveScalarType` enum after I 
remove the multi-level list changes


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