mapleFU commented on PR #46372:
URL: https://github.com/apache/arrow/pull/46372#issuecomment-2887148562
@wgtmac I'm think of keep some elements inside the `VariantValue`, the way
below adds 24B or 28B overheads on `VariantValue`. It adds some cost when
constructing the `VariantValue` but maybe more friendly for user.
```
VariantBasicType type_;
uint32_t num_elements_; // exists when type_ is array or object
uint32_t id_size_; // exists when type_ is object
uint32_t offset_size_; // exists when type_ is array or object
uint32_t id_start_offset_; // exists when type_ is object
uint32_t offset_start_offset_; // exists when type_ is array or object
uint32_t data_start_offset_; // exists when type_ is array or object
```
Another problem is the error style for decoding tools. If we just leave
VariantValue/VariantMetadata a tool in parquet, and uses it in arrow compute or
extension type, we also needs cast the handling value to status...
--
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]
