eclipse-zenoh / zenoh

zenoh unifies data in motion, data in-use, data at rest and computations. It carefully blends traditional pub/sub with geo-distributed storages, queries and computations, while retaining a level of time and space efficiency that is well beyond any of the mainstream stacks.
https://zenoh.io
Other
1.44k stars 151 forks source link

rework list of predefined encodings #1507

Open milyin opened 4 hours ago

milyin commented 4 hours ago

Describe the release item

There are several predefined encodings which became ambigious after serialization rework in https://github.com/eclipse-zenoh/zenoh/issues/1474. The main reason of ambiguity is that the prefix ZENOH_ for encoding doesn't say anything is serialization used or not.

E.g. is ZENOH_STRING means that ZBytes::to_string() was used or string was serialized into ZBytes. The difference is important: in second case the string is additionally prefixed by length. Same for ZENOH_BYTES value: is it ZBytes::to_bytes() or serialization of vector of u8. Difference is the same: just to_bytes() doesn't add any prefix

The proposal is to

milyin commented 4 hours ago

@wyfo @kydos if this that you finally decided about this issue?