You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Kenneth Knowles (JIRA)" <ji...@apache.org> on 2017/01/27 03:52:24 UTC

[jira] [Created] (BEAM-1327) Replace OutputTimeFn with enum

Kenneth Knowles created BEAM-1327:
-------------------------------------

             Summary: Replace OutputTimeFn with enum
                 Key: BEAM-1327
                 URL: https://issues.apache.org/jira/browse/BEAM-1327
             Project: Beam
          Issue Type: Bug
          Components: sdk-java-core
            Reporter: Kenneth Knowles
            Assignee: Kenneth Knowles


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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)