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

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

commit b166cccdd90a40f40364d09cd0f0675e31565a91
Author: amashenkov <[email protected]>
AuthorDate: Thu Jun 13 19:16:51 2024 +0300

    fixup! Add partition column.
---
 .../ignite/internal/sql/engine/schema/SqlSchemaManagerImpl.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/schema/SqlSchemaManagerImpl.java
 
b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/schema/SqlSchemaManagerImpl.java
index ca92b16f44..3beadff444 100644
--- 
a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/schema/SqlSchemaManagerImpl.java
+++ 
b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/schema/SqlSchemaManagerImpl.java
@@ -249,19 +249,19 @@ public class SqlSchemaManagerImpl implements 
SqlSchemaManager {
         }
 
         // Add virtual column.
-        colDescriptors.add(createPartitionVirtualColumn(colDescriptors));
+        colDescriptors.add(createPartitionVirtualColumn(columns.size()));
 
         return new TableDescriptorImpl(colDescriptors, distribution);
     }
 
-    private static ColumnDescriptorImpl 
createPartitionVirtualColumn(List<ColumnDescriptor> colDescriptors) {
+    private static ColumnDescriptorImpl createPartitionVirtualColumn(int 
logicalIndex) {
         return new ColumnDescriptorImpl(
                 Commons.PART_COL_NAME,
                 false,
                 true,
                 true,
                 true,
-                colDescriptors.size(),
+                logicalIndex,
                 NativeTypes.INT32,
                 DefaultValueStrategy.DEFAULT_NULL,
                 null

Reply via email to