xtern commented on code in PR #6274:
URL: https://github.com/apache/ignite-3/pull/6274#discussion_r2215985352
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgniteSqlValidator.java:
##########
@@ -128,6 +128,7 @@ public class IgniteSqlValidator extends SqlValidatorImpl {
public static final int MAX_LENGTH_OF_ALIASES = 256;
public static final int DECIMAL_DYNAMIC_PARAM_PRECISION = 28;
public static final int DECIMAL_DYNAMIC_PARAM_SCALE = 6;
+ private static final int TIME_DYNAMIC_PARAM_PRECISION = 6;
Review Comment:
I set the value to `6` only because TIMESTAMP has a default precision of `6`.
Maybe it's worth setting the value to `9` (for the future). Because it
doesn't affect any resources.
If no one is against it, I'll change it to `9` a little later.
--
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
