Skip to content
This repository was archived by the owner on Oct 5, 2021. It is now read-only.

Commit cc98849

Browse files
F2006Alexander Patrikalakis
authored andcommitted
Increased the delay time for tests mediumSendReceiveSerial, testMultipleReadersOnSingleLog and testMultipleReadersOnSingleLogSerial
Added author info
1 parent a07fcde commit cc98849

File tree

1 file changed

+15
-12
lines changed

1 file changed

+15
-12
lines changed

src/test/java/com/amazon/janusgraph/diskstorage/dynamodb/MultiDynamoDBLogTest.java

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,10 @@
4242
import com.google.common.base.Preconditions;
4343

4444
/**
45-
*
46-
* @author Alexander Patrikalakis
47-
*
45+
*
46+
* @author Alexander Patrikalakis
47+
* @author Johan Jacobs
48+
*
4849
*/
4950
@Category({ MultiIdAuthorityLogStoreCategory.class, MultipleItemTestCategory.class })
5051
public class MultiDynamoDBLogTest extends AbstractDynamoDBLogTest {
@@ -62,6 +63,14 @@ public MultiDynamoDBLogTest() throws Exception {
6263
* testMultipleReadersOnSingleLog
6364
* testMultipleReadersOnSingleLogSerial
6465
*
66+
* https://github.com/awslabs/dynamodb-titan-storage-backend/issues/115
67+
* Related to issue 160 above, increasing the delay time for each of the following tests
68+
* allows the tests to pass. Investigation is needed to understand what the effect of increasing the delay
69+
* has to allow the tests to pass:
70+
* mediumSendReceiveSerial
71+
* testMultipleReadersOnSingleLog
72+
* testMultipleReadersOnSingleLogSerial
73+
*
6574
* BEGIN code copied from:
6675
* https://github.com/JanusGraph/janusgraph/blob/v0.1.0/janusgraph-test/src/main/java/org/janusgraph/diskstorage/log/LogTest.java#L50
6776
*/
@@ -70,23 +79,17 @@ public MultiDynamoDBLogTest() throws Exception {
7079
@Override
7180
@Test
7281
public void mediumSendReceiveSerial() throws Exception {
73-
//TODO investigate
74-
// https://github.com/awslabs/dynamodb-titan-storage-backend/issues/115
75-
// simpleSendReceiveMine(2000,1, LONGER_TIMEOUT_MS);
82+
simpleSendReceiveMine(2000,50, LONGER_TIMEOUT_MS);
7683
}
7784
@Override
7885
@Test
7986
public void testMultipleReadersOnSingleLog() throws Exception {
80-
//TODO investigate
81-
// https://github.com/awslabs/dynamodb-titan-storage-backend/issues/115
82-
// sendReceiveMine(4, 2000, 5, false, LONGER_TIMEOUT_MS);
87+
sendReceiveMine(4, 2000, 50, false, LONGER_TIMEOUT_MS);
8388
}
8489
@Override
8590
@Test
8691
public void testMultipleReadersOnSingleLogSerial() throws Exception {
87-
//TODO investigate
88-
// https://github.com/awslabs/dynamodb-titan-storage-backend/issues/115
89-
// sendReceiveMine(4, 2000, 5, true, LONGER_TIMEOUT_MS);
92+
sendReceiveMine(4, 2000, 50, true, LONGER_TIMEOUT_MS);
9093
}
9194
private void simpleSendReceiveMine(int numMessages, int delayMS, long timeoutMS) throws Exception {
9295
sendReceiveMine(1, numMessages, delayMS, true, timeoutMS);

0 commit comments

Comments
 (0)