timoninmaxim commented on a change in pull request #9540:
URL: https://github.com/apache/ignite/pull/9540#discussion_r744868130
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/cache/query/index/sorted/IndexKeyDefinition.java
##########
@@ -70,4 +83,17 @@ public boolean validate(IndexKey key) {
return idxType == key.type();
}
+
+ /** {@inheritDoc} */
+ @Override public void writeExternal(ObjectOutput out) throws IOException {
+ // Send only required info for using in MergeSort algorithm.
+ out.writeInt(idxType);
+ out.writeInt(order.sortOrder().ordinal());
+ }
+
+ /** {@inheritDoc} */
+ @Override public void readExternal(ObjectInput in) throws IOException,
ClassNotFoundException {
Review comment:
We skip `precision` and `nulls order` because they aren't required for
MergeSort algorithm to avoid sending useless data.
--
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]
