Change in asterixdb[master]: ASTERIXDB-1126 Correct comparators and type traits for exter...

["abdullah alamoudi (Code Review)" <do...@asterixdb.incubator.apache.org>]

Hello Pouria Pirzadeh, Ian Maxon, Jenkins,

I'd like you to reexamine a change.  Please visit

    https://asterix-gerrit.ics.uci.edu/455

to look at the new patch set (#3).

Change subject: ASTERIXDB-1126 Correct comparators and type traits for external index
......................................................................

ASTERIXDB-1126 Correct comparators and type traits for external index

Turned out, the fix for ASTERIXDB-1115 introduced this bug. the reason is that it tries
to get the data type for the primary key from the record type definition. In case of external
data, the type definition doesn't contain information about primary keys and we should get
them based on the data input format. A test case was added as well.

Change-Id: I71d924d7e2b7a7e6c752bc97679e612946afc17c
---
M asterix-app/pom.xml
M asterix-app/src/test/java/org/apache/asterix/test/runtime/HDFSCluster.java
M asterix-app/src/test/java/org/apache/asterix/test/runtime/RepeatedTest.java
M asterix-common/src/test/java/org/apache/asterix/test/aql/TestsUtils.java
M asterix-installer/pom.xml
M asterix-installer/src/test/java/org/apache/asterix/installer/test/AsterixClusterLifeCycleIT.java
M asterix-installer/src/test/java/org/apache/asterix/installer/transaction/DmlRecoveryIT.java
M asterix-installer/src/test/java/org/apache/asterix/installer/transaction/RecoveryIT.java
A asterix-installer/src/test/resources/transactionts/queries/query_after_restart/external_index/external_index.1.script.aql
A asterix-installer/src/test/resources/transactionts/queries/query_after_restart/external_index/external_index.2.ddl.aql
A asterix-installer/src/test/resources/transactionts/queries/query_after_restart/external_index/external_index.3.script.aql
A asterix-installer/src/test/resources/transactionts/queries/query_after_restart/external_index/external_index.4.query.aql
A asterix-installer/src/test/resources/transactionts/queries/query_after_restart/external_index/external_index.5.script.aql
A asterix-installer/src/test/resources/transactionts/results/query_after_restart/external_index/external_index.1.adm
A asterix-installer/src/test/resources/transactionts/scripts/query_after_restart/external_index/create_and_start.sh
A asterix-installer/src/test/resources/transactionts/scripts/query_after_restart/external_index/stop_and_delete.sh
A asterix-installer/src/test/resources/transactionts/scripts/query_after_restart/external_index/stop_and_start.sh
M asterix-installer/src/test/resources/transactionts/testsuite.xml
M asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java
19 files changed, 532 insertions(+), 364 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/55/455/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/455
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I71d924d7e2b7a7e6c752bc97679e612946afc17c
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <ba...@gmail.com>
Gerrit-Reviewer: Chris Hillery <ce...@lambda.nu>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Pouria Pirzadeh <po...@gmail.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Young-Seok Kim <ki...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>