datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard
verified publisher icon apache

Simplify the serialization of `ScalarValue::List`

Open alamb opened this issue 3 years ago • 0 comments

Draft as it builds on https://github.com/apache/arrow-datafusion/pull/3536 and doesn't pass all tests

Which issue does this PR close?

Re https://github.com/apache/arrow-datafusion/issues/3531

Rationale for this change

While working on https://github.com/apache/arrow-datafusion/issues/3531 I found the ScalarValue serialization code quite challenging to understand as it did not match the structure of ScalarValue.

I also hope a final PR in this series to remove PrimitiveScalarType but I didn't quite make it today and ran out of time

What changes are included in this PR?

  1. Remove special case null handling for ScalarValue::List (it is already covered)

Are there any user-facing changes?

No

alamb avatar Sep 20 '22 16:09 alamb