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

Reply via email to