[BEAM-1205] Auto set "enableAbandonedNodeEnforcement" in TestPipeline

• Type: Improvement
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: 0.6.0
• Component/s: sdk-java-core
• Labels: Done backward-incompatible
• Assignee: Stas Levin
• Reporter: Stas Levin
• Created: 22/Dec/16 12:01
• Updated: 06/Oct/20 21:56
• Resolved: 16/Feb/17 10:21

Description

At the moment one has to manually set enableAbandonedNodeEnforcement(false) in tests that do not run the TestPipeline, otherwise one gets an AbandonedNodeException on account of having nodes that were not run.

This could probably be auto detected using the RunnableOnService and NeedsRunner annotations, the presence of which indicates a given test does indeed use a runner.

Essentially we need to check if RunnableOnService / NeedsRunner are present on a given test and if so set enableAbandonedNodeEnforcement(true), otherwise set enableAbandonedNodeEnforcement(false).

tgroh, kenn

Issue Links

• links to: GitHub Pull Request #1706