commit 41fae6a4ec144d7e74556e3c768ddcbf73b70976
author Peeyush Gupta <peeyush.gupta@couchbase.com> Mon May 15 11:39:25 2023 -0700
committer Peeyush Gupta <peeyush.gupta@couchbase.com> Tue May 16 16:02:40 2023 +0000
tree ff08c5e893a4ae59b1d5338d90189e75437b86ed
parent 43b0b3ed28baca75e6b9f47ac0b72d95a837fa7d

[NO-ISSUE][MTD] Incorrect deserailization of dataset primary key types

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
While deserializing dataset details, all primary key types were
incorrectly set to string.

Change-Id: I20636cb8535709f607e8748e1fdd9383ab269cf7
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17532
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Murtadha Hubail <mhubail@apache.org>
Reviewed-by: Murtadha Hubail <mhubail@apache.org>

asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java

diff --git a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java
index e659b8f..70dfde1 100644
--- a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java
+++ b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java
@@ -149,7 +149,6 @@
                         .getValueByPos(MetadataRecordTypes.INTERNAL_DETAILS_ARECORD_PARTITIONKEY_FIELD_INDEX))
                                 .getCursor();
                 List<List<String>> partitioningKey = new ArrayList<>();
-                List<IAType> partitioningKeyType = new ArrayList<>();
 
                 while (cursor.next()) {
                     AOrderedList fieldNameList = (AOrderedList) cursor.get();
@@ -159,7 +158,6 @@
                         nestedFieldName.add(((AString) nestedFieldNameCursor.get()).getStringValue());
                     }
                     partitioningKey.add(nestedFieldName);
-                    partitioningKeyType.add(BuiltinType.ASTRING);
                 }
 
                 // Check if there is a primary key types field
@@ -223,7 +221,7 @@
                 }
 
                 datasetDetails = new InternalDatasetDetails(fileStructure, partitioningStrategy, partitioningKey,
-                        partitioningKey, keyFieldSourceIndicator, partitioningKeyType, autogenerated,
+                        partitioningKey, keyFieldSourceIndicator, primaryKeyTypes, autogenerated,
                         filterSourceIndicator, filterField);
                 break;
             }