Steven Jacobs has posted comments on this change. Change subject: [ASTERIXDB-1911][HYR,RT,CLUS] Fixes and Improvements for PreDistributed Jobs ......................................................................
Patch Set 22: (4 comments) Addressed Comments and pushing a new change https://asterix-gerrit.ics.uci.edu/#/c/2045/22/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/DeployedJobService.java File asterixdb/asterix-active/src/main/java/org/apache/asterix/active/DeployedJobService.java: Line 46: private static final int poolSize = 1; > please address this. Done https://asterix-gerrit.ics.uci.edu/#/c/2045/22/hyracks-fullstack/hyracks/hyracks-examples/hyracks-integration-tests/src/test/java/org/apache/hyracks/tests/integration/DeployedJobSpecsTest.java File hyracks-fullstack/hyracks/hyracks-examples/hyracks-integration-tests/src/test/java/org/apache/hyracks/tests/integration/DeployedJobSpecsTest.java: Line 129: verify(cc, Mockito.timeout(5000).times(4)).getDeployedJobSpecStore(); > use a single private static final int to replace this and all following occ Done Line 130: verify(nc1, Mockito.timeout(5000).times(2)).storeActivityClusterGraph(any(), any()); > Is there any particular reason to chose 4,2,8,12 for the times? There are 2 specifications and 2 NCs. All of these just come from the number of times that these things should happen based on these 2 numbers. Line 173: Thread.sleep(10000); > do we have more deterministic method instead of sleep? this is likely to fa Actually we don't need this at all anymore. Since we make a completely new job every time, the job won't even be lingering for the sweeper to try to clean up. I'll remove this. -- To view, visit https://asterix-gerrit.ics.uci.edu/2045 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f493c1fa977d07dfe8a875f9ebe9515d01d1473 Gerrit-PatchSet: 22 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Steven Jacobs <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin <[email protected]> Gerrit-Reviewer: Ildar Absalyamov <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Steven Jacobs <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Xikui Wang <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: Yes
