risingwave icon indicating copy to clipboard operation
risingwave copied to clipboard

Published 20 hours ago • verified publisher icon risingwavelabs

bug: migrate from `from_proto_bytes` to value encoding (at least for struct/list)

Open fuyufjh opened this issue 1 year ago • 1 comments

from_proto_bytes is not well maintained and is majorly used to encode Literal data in proto, but our List and Struct data types use them as their value encoding, which may cause some bugs.

Furthermore, we may consider removing from_proto_bytes completely because it's so error-prone。

fuyufjh avatar Aug 16 '22 09:08 fuyufjh

Any updates?

fuyufjh avatar Aug 26 '22 02:08 fuyufjh

removed for struct/list by #5880

waruto210 avatar Oct 17 '22 13:10 waruto210

done by #5961

waruto210 avatar Oct 21 '22 10:10 waruto210