[BEAM-1327] Replace OutputTimeFn with enum

• Type: New Feature
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: sdk-java-core
• Labels: Done backward-incompatible
• Assignee: Kenneth Knowles
• Reporter: Kenneth Knowles
• Created: 27/Jan/17 03:52
• Updated: 06/Oct/20 21:57
• Resolved: 02/May/17 20:32

Description

The class OutputTimeFn is overkill for a Fn API crossing. There are only three sensible values known: MIN, MAX, EOW. The interface is right for implementing these, but the full class is left over from the days when there was little cost to shipping new kinds of fns. An enum is concise.

This can be done "mostly" backwards compatibly with legacy adapters in place, but might be less confusing without them.

Issue Links

• relates to: BEAM-115 Beam Runner API
• links to: GitHub Pull Request #2683
• links to: GitHub Pull Request #2741