accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1748) Restore pause in 1.4.x, 1.5.x PermissionsTest
Date Wed, 09 Oct 2013 22:40:42 GMT

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

Christopher Tubbs commented on ACCUMULO-1748:
---------------------------------------------

{quote}I'm not a big fan of including arbitrary sleeps; IMO, we should try to write tests
in a way that is reliable.{quote}
+1

> Restore pause in 1.4.x, 1.5.x PermissionsTest
> ---------------------------------------------
>
>                 Key: ACCUMULO-1748
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1748
>             Project: Accumulo
>          Issue Type: Test
>          Components: test
>    Affects Versions: 1.4.4, 1.5.0
>            Reporter: Bill Havanki
>            Priority: Trivial
>              Labels: newbie
>
> While running functional tests for Accumulo 1.4.3 I see sporadic failure in the SystemPermissionTest
test.
> {noformat}
> Caused by: java.lang.IllegalStateException: Should be able to remove a table property
> {noformat}
> I found that in earlier versions of Accumulo there was a pause of 100 ms between the
call to remove the property and the check that it is gone. The pause was removed early this
year, see line 237 of:
> http://svn.apache.org/viewvc/accumulo/trunk/test/src/main/java/org/apache/accumulo/test/functional/PermissionsTest.java?r1=1439896&r2=1442284&pathrev=1449794&diff_format=h
> The comment with the pause was: "Surprised we got away without pausing before." :)
> I was unable to determine from the issues associated with the commit why the pause was
removed. So, I'd like it to be put back, in both the 1.4.x and 1.5.x branches.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message