leerho commented on code in PR #162:
URL:
https://github.com/apache/datasketches-website/pull/162#discussion_r1508343150
##########
docs/Architecture/LargeScale.md:
##########
@@ -33,7 +33,9 @@ layout: doc_page
* The C++ Core is extended using the python binding library
[pybind11](https://github.com/pybind/pybind11) enabling high performance
operation from Python.
### Cross Language Binary Compatibility
-* Sketches serialized from C++ or Python can be interpreted by compatible Java
sketches and visa versa.
+* Sketches serialized from C++ or Python can be interpreted by compatible Java
sketches and visa versa.
+
+* All sketches have a serialized form which is able to be deserialized by any
version of the library since the sketch was introduced.
Review Comment:
Thank you for volunteering this! However, the statement is a bit too
general. Because of differences between languages, there may be sketches in
C++, for example, that don't exist in Java yet. Or visa-versa. So we need a
little more precision in the wording. For example:
"All sketches have a serialized form which is able to be deserialized by the
same or later version of the same sketch in the same language of the library
since the sketch was introduced. Deserialization across languages and across
time for compatible sketches is a bit trickier since there may be a time-lag
between languages when a specific sketch was introduced."
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
