pitrou commented on code in PR #45272:
URL: https://github.com/apache/arrow/pull/45272#discussion_r2190785054
##########
cpp/src/arrow/array/data.cc:
##########
@@ -495,13 +495,15 @@ void ArraySpan::FillFromScalar(const Scalar& value) {
if (type_id == Type::LIST) {
const auto& list_scalar = checked_cast<const ListScalar&>(value);
this->buffers[1] = OffsetsForScalar(list_scalar.scratch_space_,
sizeof(int32_t));
+ this->length = list_scalar.value->length();
Review Comment:
> I think this is causing issues elsewhere for now, but before diving into
them further I want to check if we have an invariant that the length of any
scalar is always equal to 1.
I think that's the only thing that makes sense, yes.
--
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]
