It would make more sense to me to default "columnInfo->length = NValue::getTupleStorageSize(type);" HERE instead of relying on code in various places -- the caller that sets length for scalar types, the non-inline cases below handling object types.

Similarly, offset can be defaulted to this same value and reset below for inlines.