Michael Luckey created BEAM-7197:
------------------------------------
Summary: ParDoLifecycleTest: exeption 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
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)
