[BEAM-14027] Java PreCommit flaky org.apache.beam.sdk.transforms/ParDoLifecycleTest#testTeardownCalledAfterExceptionInProcessElementStateful

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: test-failures
• Assignee: Reuven Lax
• Reporter: Luke Cwik
• Created: 02/Mar/22 17:07
• Updated: 09/Mar/22 17:43
• Resolved: 09/Mar/22 17:43

Description

Looks like https://github.com/apache/beam/pull/16727 is using the static map concurrently which is not thread safe.

Causing failures like:

java.util.ConcurrentModificationException
	at java.util.HashMap$HashIterator.nextNode(HashMap.java:1469)
	at java.util.HashMap$ValueIterator.next(HashMap.java:1498)
	at java.util.AbstractCollection.toString(AbstractCollection.java:461)
	at java.lang.String.valueOf(String.java:2994)
	at java.lang.StringBuilder.append(StringBuilder.java:131)
	at org.apache.beam.sdk.transforms.ParDoLifecycleTest$ExceptionThrowingFn.validate(ParDoLifecycleTest.java:392)
	at org.apache.beam.sdk.transforms.ParDoLifecycleTest$ExceptionThrowingFn.access$300(ParDoLifecycleTest.java:356)
	at org.apache.beam.sdk.transforms.ParDoLifecycleTest.testTeardownCalledAfterExceptionInProcessElementStateful(ParDoLifecycleTest.java:269)

See failure in:
https://ci-beam.apache.org/job/beam_PreCommit_Java_Commit/21291/testReport/junit/org.apache.beam.sdk.transforms/ParDoLifecycleTest/testTeardownCalledAfterExceptionInProcessElementStateful/

Issue Links

• links to: GitHub Pull Request #16991