You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Stas Levin (JIRA)" <ji...@apache.org> on 2017/01/01 15:23:58 UTC
[jira] [Updated] (BEAM-1223) Add static factory methods for
Sum.[*]Fn classes
[ https://issues.apache.org/jira/browse/BEAM-1223?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Stas Levin updated BEAM-1223:
-----------------------------
Description:
{{Sum.SumDoubleFn}}, {{SumIntegerFn}} and {{SumLongFn}} are not using the {{X.of()}} or {{X.from()}} or other instance creation via static method patterns that are ubiquitous in Beam.
Following a discussion on the dev list, it would be great to preserve a consistent look and feel and change the creation patterns for these classes to something like {{SumFn.ofLong()}} etc.
See also the corresponding [dev list thread|https://lists.apache.org/thread.html/5d8e905ee49b116d13811c2a96da65eeb44ab7c002870f50936ee1ad@%3Cdev.beam.apache.org%3E].
was:
{{Sum.SumDoubleFn}}, {{SumIntegerFn}} and {{SumLongFn}} are not using the {{X.of()}} or {{X.from()}} or other instance creation via static method patterns that are ubiquitous in Beam.
Following a discussion on the dev list, it would be great to preserve a consistent look and feel and change the creation patterns for these classes to something like {{SumFn.ofLong()}} etc.
> Add static factory methods for Sum.[*]Fn classes
> ------------------------------------------------
>
> Key: BEAM-1223
> URL: https://issues.apache.org/jira/browse/BEAM-1223
> Project: Beam
> Issue Type: Improvement
> Components: sdk-java-core
> Reporter: Stas Levin
> Assignee: Stas Levin
> Priority: Minor
>
> {{Sum.SumDoubleFn}}, {{SumIntegerFn}} and {{SumLongFn}} are not using the {{X.of()}} or {{X.from()}} or other instance creation via static method patterns that are ubiquitous in Beam.
> Following a discussion on the dev list, it would be great to preserve a consistent look and feel and change the creation patterns for these classes to something like {{SumFn.ofLong()}} etc.
> See also the corresponding [dev list thread|https://lists.apache.org/thread.html/5d8e905ee49b116d13811c2a96da65eeb44ab7c002870f50936ee1ad@%3Cdev.beam.apache.org%3E].
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)