superminkfan commented on code in PR #12423:
URL: https://github.com/apache/ignite/pull/12423#discussion_r2450396994
##########
modules/core/src/main/java/org/apache/ignite/configuration/SqlConfiguration.java:
##########
@@ -51,7 +51,12 @@ public class SqlConfiguration {
/** SQL plan history size. */
private int sqlPlanHistSize = -1;
- /** Enable validation of key & values against sql schema. */
+ /**
+ * Enables validation of cache keys and values against the SQL schema that
describes the cache.
+ * When turned on, Ignite checks runtime types, nullability and
length/precision constraints before
Review Comment:
Clarified the validationEnabled field, getter, and setter Javadoc so they
explain that Ignite already enforces declared nullability and precision
constraints, and that enabling the flag adds runtime type checks (with the
resulting exception types noted).
Confirmed through QueryTypeDescriptorImpl that properties with NOT NULL or
precision settings are validated even when type validation is disabled, while
enabling the flag extends checks to all column types.
--
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]
