Young-Seok Kim has posted comments on this change. Change subject: ASTERIXDB-1538: fix the issue where LSMRTree index instance is created during the recovery process. ......................................................................
Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1010/1/asterixdb/asterix-installer/src/test/resources/transactionts/queries/recover_after_abort/primary_plus_rtree_index_insert_and_delete/primary_plus_rtree_index_insert_and_delete.4.txneu.aql File asterixdb/asterix-installer/src/test/resources/transactionts/queries/recover_after_abort/primary_plus_rtree_index_insert_and_delete/primary_plus_rtree_index_insert_and_delete.4.txneu.aql: Line 69: /* Finally, all we have should be the rows with the rowid between 50 and 100 */ Let's remove this last sentence. This is not guaranteed since while the row 60 is being inserted into a partition and concurrently another duplicated key row 61 is inserted into another partition and throws duplicated exception before the row 60 is committed. -- To view, visit https://asterix-gerrit.ics.uci.edu/1010 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I701a7ad748ff8982534a132a01f5a1ae77c952fd Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Taewoo Kim <wangs...@yahoo.com> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Young-Seok Kim <kiss...@gmail.com> Gerrit-HasComments: Yes