[
https://issues.apache.org/jira/browse/BEAM-7197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ismaël Mejía updated BEAM-7197:
-------------------------------
Summary: ParDoLifecycleTest: exception throwing tests broken (was:
ParDoLifecycleTest: exeption throwing tests broken)
> ParDoLifecycleTest: exception throwing tests broken
> ---------------------------------------------------
>
> Key: BEAM-7197
> URL: https://issues.apache.org/jira/browse/BEAM-7197
> Project: Beam
> Issue Type: Bug
> Components: sdk-java-core
> Reporter: Michael Luckey
> Assignee: Michael Luckey
> Priority: Major
> Time Spent: 3h 40m
> Remaining Estimate: 0h
>
> ParDoLifecycleTest implements tests to assert that DoFn are tore down after
> another lifecycle method throw an exception.
>
> The implementation uses a static AtomicBoolean for assertions [1].
> Unfortunately, this is never reset which results in that boolean being true
> after the first test which happens to correctly call teardown on exception.
> Failures for tests executed after are essentially hidden.
> This can be seen e.g. by
> {code:bash}
> ./gradlew -p runners/spark/ validatesRunnerBatch --tests
> org.apache.beam.sdk.transforms.ParDoLifecycleTest.testTeardownCalledAfterExceptionInStartBundle
>
> {code}
> [1]
> https://github.com/apache/beam/blob/master/sdks/java/core/src/test/java/org/apache/beam/sdk/transforms/ParDoLifecycleTest.java#L407-L412
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
