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

David Capwell commented on CASSANDRA-15313:
-------------------------------------------

bq. Reverting CASSANDRA-15555 fixes corruptionCausesFailure with seed 
71671740653044L for me

Sorry I don't fully follow, can you elaborate [~spod]?  There are 3 issues with 
the test/feature (may have missed something, going off memory).

1) corruption can cause lz4 to crash the JVM.  This was fixed in 
CASSANDRA-15556 by using the "safe" methods rather than "fast"
2) corrupted lz4 stream may not fail and may produce output != input; this is 
still an issue and the tests fail periodically with this.
3) generators generated too much garbage.  CASSANDRA-15555 switched to fixed 
memory and switched from strings (charset depends on the test environment since 
it doesn't define which charset to use) to raw bytes.  Given the fact the 
generated data changed the seeds which failed before no longer fail, and the 
seeds that fail now did not fail with the old generators; both generators were 
able to reproduce #2, but with different seeds.

> Fix flaky - ChecksummingTransformerTest - 
> org.apache.cassandra.transport.frame.checksum.ChecksummingTransformerTest
> -------------------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-15313
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15313
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/dtest
>            Reporter: Vinay Chella
>            Assignee: Brandon Williams
>            Priority: Normal
>             Fix For: 4.0-alpha
>
>         Attachments: CASSANDRA-15313-hack.patch
>
>
> During the recent runs, this test appears to be flaky.
> Example failure: 
> [https://circleci.com/gh/vinaykumarchella/cassandra/459#tests/containers/94]
> corruptionCausesFailure-compression - 
> org.apache.cassandra.transport.frame.checksum.ChecksummingTransformerTest
> {code:java}
> java.lang.OutOfMemoryError: GC overhead limit exceeded
>       at java.nio.HeapByteBuffer.<init>(HeapByteBuffer.java:57)
>       at java.nio.ByteBuffer.allocate(ByteBuffer.java:335)
>       at org.quicktheories.impl.Precursor.<init>(Precursor.java:17)
>       at 
> org.quicktheories.impl.ConcreteDetachedSource.<init>(ConcreteDetachedSource.java:8)
>       at 
> org.quicktheories.impl.ConcreteDetachedSource.detach(ConcreteDetachedSource.java:23)
>       at org.quicktheories.generators.Retry.generate(CodePoints.java:51)
>       at 
> org.quicktheories.generators.Generate.lambda$intArrays$10(Generate.java:190)
>       at 
> org.quicktheories.generators.Generate$$Lambda$17/1847008471.generate(Unknown 
> Source)
>       at org.quicktheories.core.DescribingGenerator.generate(Gen.java:255)
>       at org.quicktheories.core.Gen.lambda$map$0(Gen.java:36)
>       at org.quicktheories.core.Gen$$Lambda$20/71399214.generate(Unknown 
> Source)
>       at org.quicktheories.core.Gen.lambda$map$0(Gen.java:36)
>       at org.quicktheories.core.Gen$$Lambda$20/71399214.generate(Unknown 
> Source)
>       at org.quicktheories.core.Gen.lambda$mix$10(Gen.java:184)
>       at org.quicktheories.core.Gen$$Lambda$45/802243390.generate(Unknown 
> Source)
>       at org.quicktheories.core.Gen.lambda$flatMap$5(Gen.java:93)
>       at org.quicktheories.core.Gen$$Lambda$48/363509958.generate(Unknown 
> Source)
>       at 
> org.quicktheories.dsl.TheoryBuilder4.lambda$prgnToTuple$12(TheoryBuilder4.java:188)
>       at 
> org.quicktheories.dsl.TheoryBuilder4$$Lambda$40/2003496028.generate(Unknown 
> Source)
>       at org.quicktheories.core.DescribingGenerator.generate(Gen.java:255)
>       at org.quicktheories.core.FilteredGenerator.generate(Gen.java:225)
>       at org.quicktheories.core.Gen.lambda$map$0(Gen.java:36)
>       at org.quicktheories.core.Gen$$Lambda$20/71399214.generate(Unknown 
> Source)
>       at org.quicktheories.impl.Core.generate(Core.java:150)
>       at org.quicktheories.impl.Core.shrink(Core.java:103)
>       at org.quicktheories.impl.Core.run(Core.java:39)
>       at org.quicktheories.impl.TheoryRunner.check(TheoryRunner.java:35)
>       at org.quicktheories.dsl.TheoryBuilder4.check(TheoryBuilder4.java:150)
>       at 
> org.quicktheories.dsl.TheoryBuilder4.checkAssert(TheoryBuilder4.java:162)
>       at 
> org.apache.cassandra.transport.frame.checksum.ChecksummingTransformerTest.corruptionCausesFailure(ChecksummingTransformerTest.java:87)
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to