ibessonov commented on code in PR #6161:
URL: https://github.com/apache/ignite-3/pull/6161#discussion_r2177372372


##########
modules/network/src/main/java/org/apache/ignite/internal/network/direct/stream/DirectByteBufferStreamImplV1.java:
##########
@@ -73,6 +74,23 @@
  * {@link DirectByteBufferStream} implementation.
  */
 public class DirectByteBufferStreamImplV1 implements DirectByteBufferStream {
+    /**
+     * There's a binary incompatibility between Apache Ignite 3.0 and later 
versions due to the bug in 3.0. If someone wants to upgrade from
+     * 3.0 to any other version on AARCH64 architecture or other architecture 
that was considered "not unaligned" in Ignite 3.0, they need
+     * to set this system property to {@code true}.
+     *
+     * @see #writeUuidLong(long, long)
+     * @see #readUuidLong(long)
+     * @see #unaligned30()
+     */
+    private static final String IGNITE_30_3X_COMPATIBILITY_PROPERTY = 
"IGNITE_30_3X_UUIDS_SERIALIZATION_COMPATIBILITY";

Review Comment:
   > how will the user understand that this is the problem?
   
   We will have to document it
   
   > Maybe we already write somewhere which PDS on which version the node 
directory was created and then write a warning.
   
   Not going to work, most likely, for a complicated reason that I might 
explain later if you ask. It will require a lot of thinking to make this 
approach work



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to