geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GEODE-186) ClientServerTransactionDUnitTest.testSuspendTimeout takes too long to run
Date Thu, 06 Aug 2015 17:12:05 GMT

    [ https://issues.apache.org/jira/browse/GEODE-186?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660362#comment-14660362
] 

ASF subversion and git services commented on GEODE-186:
-------------------------------------------------------

Commit 389e2a8cac1f2b35aa3ebfd10e8c49ae993b0dc4 in incubator-geode's branch refs/heads/feature/GEODE-137
from [~dschneider]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=389e2a8 ]

Fix GEODE-186 by removing sleeps in tests

The old test scheduled tx suspension to timeout after 1 minute.
So the test always run for at least 1 minute.
A test hook now exists that allows the test to specify
a different time unit (default is still minutes) for
tx suspension expiration.

The sleeps in a bunch of other tests were not needed
since the tx operation is synchronous. So those sleeps
have simply been removed.

A couple of sleeps in clients waiting for something to
arrive that was done on a server have been converted to
a wait since server to client distribution is async.

Reviewed by Swapnil.


> ClientServerTransactionDUnitTest.testSuspendTimeout takes too long to run
> -------------------------------------------------------------------------
>
>                 Key: GEODE-186
>                 URL: https://issues.apache.org/jira/browse/GEODE-186
>             Project: Geode
>          Issue Type: Bug
>            Reporter: Darrel Schneider
>            Assignee: Darrel Schneider
>            Priority: Minor
>
> ClientServerTransactionDUnitTest.testSuspendTimeout takes too long to run. It takes 70
seconds. It has a 70 second sleep. It is waiting for a suspended tx to expire. It should use
a test hook to change the expiration units from minutes to millis.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message