[infinispan-issues] [JBoss JIRA] (ISPN-8728) ExceptionEvictionTest.testSizeCorrectWithStateTransfer random failures

[William Burns (JIRA)]

    [ https://issues.jboss.org/browse/ISPN-8728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13526911#comment-13526911 ] 

William Burns edited comment on ISPN-8728 at 1/31/18 3:27 PM:
--------------------------------------------------------------

Weird looking at the logs it appears the amounts were correct:

{code}
15:11:39,328 TRACE (transport-thread-Test-NodeC-p67589-t6:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 216
15:11:39,329 TRACE (transport-thread-Test-NodeH-p67669-t1:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 144
15:11:39,503 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,510 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 48
15:11:39,512 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,515 TRACE (stateTransferExecutor-thread-Test-NodeC-p67590-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,523 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,525 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,530 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 72
15:11:39,531 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,561 TRACE (transport-thread-Test-NodeB-p67581-t3:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 192
15:11:39,563 TRACE (transport-thread-Test-NodeI-p67677-t5:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 216
15:11:39,613 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,614 TRACE (remote-thread-Test-NodeA-p67571-t5:[___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 120
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 96
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t4:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 168
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 144
15:11:39,617 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,636 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
{code}

As you can see the last printed out from A, B & C are
{code}
15:11:39,515 TRACE (stateTransferExecutor-thread-Test-NodeC-p67590-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,636 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
{code}

So I would expect the size to be 720 like the test asserted. I am not sure where the extra 24 is coming from




was (Author: william.burns):
Weird looking at the logs it appears the amounts were correct:

{quote}
15:11:39,328 TRACE (transport-thread-Test-NodeC-p67589-t6:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 216
15:11:39,329 TRACE (transport-thread-Test-NodeH-p67669-t1:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 144
15:11:39,503 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,510 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 48
15:11:39,512 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,515 TRACE (stateTransferExecutor-thread-Test-NodeC-p67590-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,523 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,525 TRACE (stateTransferExecutor-thread-Test-NodeI-p67678-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,530 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 72
15:11:39,531 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,561 TRACE (transport-thread-Test-NodeB-p67581-t3:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 192
15:11:39,563 TRACE (transport-thread-Test-NodeI-p67677-t5:[Topology-___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by -24 to 216
15:11:39,613 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,614 TRACE (remote-thread-Test-NodeA-p67571-t5:[___defaultcache]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 120
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 96
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t4:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 168
15:11:39,614 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 144
15:11:39,617 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 192
15:11:39,636 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t3:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 216
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
{quote}

As you can see the last printed out from A, B & C are
{quote}
15:11:39,515 TRACE (stateTransferExecutor-thread-Test-NodeC-p67590-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,636 TRACE (stateTransferExecutor-thread-Test-NodeB-p67582-t1:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
15:11:39,643 TRACE (stateTransferExecutor-thread-Test-NodeA-p67574-t2:[]) [TransactionalExceptionEvictionInterceptor] Increased exception based size by 24 to 240
{quote}

So I would expect the size to be 720 like the test asserted. I am not sure where the extra 24 is coming from



> ExceptionEvictionTest.testSizeCorrectWithStateTransfer random failures
> ----------------------------------------------------------------------
>
>                 Key: ISPN-8728
>                 URL: https://issues.jboss.org/browse/ISPN-8728
>             Project: Infinispan
>          Issue Type: Bug
>          Components: Test Suite - Core
>    Affects Versions: 9.2.0.CR1
>            Reporter: Dan Berindei
>            Assignee: William Burns
>              Labels: testsuite_stability
>             Fix For: 9.2.0.CR2
>
>         Attachments: ExceptionEvictionTest_20180129.log.gz
>
>
> {noformat}
> 15:10:01,610 ERROR (testng-Test:[]) [TestSuiteProgress] Test failed: org.infinispan.eviction.impl.ExceptionEvictionTest.testSizeCorrectWithStateTransfer[DIST_SYNC, nodeCount=3, storageType=BINARY, optimisticTransaction=true]
> java.lang.AssertionError: expected:<1920> but was:<1984>
> 	at org.testng.AssertJUnit.fail(AssertJUnit.java:59) ~[testng-6.8.8.jar:?]
> 	at org.testng.AssertJUnit.failNotEquals(AssertJUnit.java:364) ~[testng-6.8.8.jar:?]
> 	at org.testng.AssertJUnit.assertEquals(AssertJUnit.java:80) ~[testng-6.8.8.jar:?]
> 	at org.testng.AssertJUnit.assertEquals(AssertJUnit.java:170) ~[testng-6.8.8.jar:?]
> 	at org.testng.AssertJUnit.assertEquals(AssertJUnit.java:177) ~[testng-6.8.8.jar:?]
> 	at org.infinispan.eviction.impl.ExceptionEvictionTest.assertInterceptorCount(ExceptionEvictionTest.java:252) ~[test-classes/:?]
> 	at org.infinispan.eviction.impl.ExceptionEvictionTest.testSizeCorrectWithStateTransfer(ExceptionEvictionTest.java:600) ~[test-classes/:?]
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.5.0#75005)