This is an automated email from the ASF dual-hosted git repository.

amashenkov pushed a commit to branch ignite-21580
in repository https://gitbox.apache.org/repos/asf/ignite-3.git

commit 80e48e1a153d4b4609d32e5e98cf31d8fc9ca906
Author: amashenkov <andrey.mashen...@gmail.com>
AuthorDate: Wed Mar 27 21:09:35 2024 +0300

    Fix Collation pass-through for sorted map aggregate
---
 .../ignite/internal/sql/engine/rel/agg/IgniteSortAggregateBase.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/agg/IgniteSortAggregateBase.java
 
b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/agg/IgniteSortAggregateBase.java
index d763c49b95..70570999eb 100644
--- 
a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/agg/IgniteSortAggregateBase.java
+++ 
b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/agg/IgniteSortAggregateBase.java
@@ -80,7 +80,7 @@ interface IgniteSortAggregateBase extends 
TraitsAwareIgniteRel {
         ImmutableBitSet requiredKeys = ImmutableBitSet.of(required.getKeys());
         RelCollation collation;
 
-        if (getGroupSet().contains(requiredKeys)) {
+        if 
(ImmutableBitSet.range(getGroupSet().cardinality()).contains(requiredKeys)) {
             List<RelFieldCollation> newCollationFields = new 
ArrayList<>(getGroupSet().cardinality());
             newCollationFields.addAll(required.getFieldCollations());
 

Reply via email to