alex-plekhanov commented on a change in pull request #9671:
URL: https://github.com/apache/ignite/pull/9671#discussion_r796634352
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/rel/IgniteIndexScan.java
##########
@@ -39,13 +41,18 @@
/** */
private final long sourceId;
+ /** */
+ private final RelCollation idxCollation;
+
/**
* Constructor used for deserialization.
*
* @param input Serialized representation.
*/
public IgniteIndexScan(RelInput input) {
- super(changeTraits(input, IgniteConvention.INSTANCE));
+ super(changeTraits(input, IgniteConvention.INSTANCE,
input.getCollation()));
Review comment:
> Perhaps, it's better to store index collation in the "collation" field
and restore output collation on remote nodes by index collation and projects.
Hmm, but in this case sort node will be inserted even if it's not required
(for example, when we just filter values by index), looks like it's not an
optimal solution.
--
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]
