You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@beam.apache.org by "Luke Cwik (JIRA)" <ji...@apache.org> on 2016/12/22 19:20:58 UTC

[jira] [Resolved] (BEAM-1208) Visibility of Sum.SumIntegerFn/Sum.SumDoubleFn/Sum.SumYYYFn should be reduced

     [ https://issues.apache.org/jira/browse/BEAM-1208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luke Cwik resolved BEAM-1208.
-----------------------------
       Resolution: Won't Fix
    Fix Version/s: Not applicable

It turns out that these are exposed so that they can be used with composed combines.

> Visibility of Sum.SumIntegerFn/Sum.SumDoubleFn/Sum.SumYYYFn should be reduced
> -----------------------------------------------------------------------------
>
>                 Key: BEAM-1208
>                 URL: https://issues.apache.org/jira/browse/BEAM-1208
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Luke Cwik
>            Assignee: Luke Cwik
>            Priority: Minor
>              Labels: beginner, documentation
>             Fix For: Not applicable
>
>
> Visibility of Sum.SumIntegerFn/Sum.SumDoubleFn/Sum.SumYYYFn should be reduced and users attention should be applied to the Sum.YYYPerKey or Sum.YYYGlobally. This can be done by moving the implementations of Sum.SumYYYFn outside of the Sum transform or the visibility of the inner classes reduced.
> Combine.java should not use Sum.SumYYYFn in their javadocs.



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