search

weld-project / weld

High-performance runtime for data analytics applications
https://www.weld.rs
BSD 3-Clause "New" or "Revised" License
2.99k stars 259 forks source link

Change serialize/deserialize to u8 #447

Closed Max-Meldrum closed 5 years ago

Max-Meldrum commented 5 years ago

I think that should be all necessary changes. But let me know if you see any missing changes.

closes #446

All serialization examples in the cpp run accordingly. Also, the weld (rust) serialization tests pass as well.

sppalkia commented 5 years ago

Looks good, thanks for adding this!