timoninmaxim commented on a change in pull request #9167:
URL: https://github.com/apache/ignite/pull/9167#discussion_r693790059
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/cache/query/index/sorted/IndexKeyDefinition.java
##########
@@ -34,11 +34,20 @@
/** Order. */
private final Order order;
+ /** Precision for variable length key types. */
+ private final long precision;
Review comment:
Yes, you're correct and actually this is my fault. Currently there is
only consumer of the IndexKeyDefinition constructor (QueryIndexDefinition). And
this logic should be moved to this class. But as it's the only place, it
doesn't affect much.
I'll fix it in other ticket I'm working now, that makes some changes in
IndexKeyDefinition.
--
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]
