Steven Jacobs has posted comments on this change. Change subject: Allow BAD jobs to update their specifications to use new indexes ......................................................................
Patch Set 9: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2620/9/asterix-bad/src/test/java/org/apache/asterix/bad/test/BADListenerTest.java File asterix-bad/src/test/java/org/apache/asterix/bad/test/BADListenerTest.java: Line 33: private static DeployedJobSpecEventListener djsel; > This test case doesn't cover what you would want to test. A good test examp I am covering such cases. Line 78 does (1), Line 79 does (2) then 5 seconds later line 78 completes and thereby completes (2). Lines 76 and 77 do (3). -- To view, visit https://asterix-gerrit.ics.uci.edu/2620 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If0a4d37a5b91063fcb1673dbfd008c140ed54ae6 Gerrit-PatchSet: 9 Gerrit-Project: asterixdb-bad Gerrit-Branch: master Gerrit-Owner: Steven Jacobs <sjaco...@ucr.edu> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Steven Jacobs <sjaco...@ucr.edu> Gerrit-Reviewer: Xikui Wang <xkk...@gmail.com> Gerrit-HasComments: Yes