Phillippko commented on code in PR #12155:
URL: https://github.com/apache/ignite/pull/12155#discussion_r2191965896
##########
modules/core/src/main/java/org/apache/ignite/internal/util/IgniteUtils.java:
##########
@@ -349,6 +349,14 @@ public abstract class IgniteUtils extends CommonUtils {
/** Ignite package. */
public static final String IGNITE_PKG = "org.apache.ignite.";
+ /**
+ * To avoid tree corruption, at least two items should fit into one page
of H2 index tree.
+ * So maximum payload size equals: P = (PS - H - 3L) / 2 - X , where P -
Payload size, PS - page size, H - page
+ * header size, L - size of the child link, X - overhead per item.
Calculated for pageSize = 1KB with
+ * KeystoreEncryptionSpi and MVCC enabled.
Review Comment:
Discussed in the ticket with @timoninmaxim , decided to just print WARN on
server nodes, use real page size and provide a constant to easily configure min
recommended number of entries in the tree. Updated the PR
--
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
