vortex icon indicating copy to clipboard operation
vortex copied to clipboard

Implement Display for ArrayMetadata

Open lwwmanning opened this issue 1 year ago • 3 comments

and remove e.g.,

// TODO(ngates): use Display for metadata

https://github.com/spiraldb/vortex/blob/0ad2232/vortex-array/src/tree.rs#L60

lwwmanning avatar Jul 22 '24 14:07 lwwmanning

@lwwmanning I'm looking through the good first issue issues and found this.

Do you have an opinion on the format? BoolMetadata at least has a validity, length, and bit_offset. I could do something like "bool ffa1 64 0" where the validity vector is encoded as a hex string?

danking avatar Aug 23 '24 16:08 danking

We should not do this right now. All of array metadata have to be audited for size and it will likely make the display completely different

robert3005 avatar Aug 23 '24 16:08 robert3005

Should wait for #396

robert3005 avatar Aug 23 '24 17:08 robert3005

done in #975

robert3005 avatar Nov 05 '24 13:11 robert3005