>From Peeyush Gupta <[email protected]>:
Peeyush Gupta has uploaded this change for review. (
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17532 )
Change subject: [NO-ISSUE][MTD] Incorrect deserailization of dataset primary
key types
......................................................................
[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
---
M
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entitytupletranslators/DatasetTupleTranslator.java
1 file changed, 18 insertions(+), 3 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/32/17532/1
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;
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17532
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I20636cb8535709f607e8748e1fdd9383ab269cf7
Gerrit-Change-Number: 17532
Gerrit-PatchSet: 1
Gerrit-Owner: Peeyush Gupta <[email protected]>
Gerrit-MessageType: newchange
