[BEAM-1223] Replace public constructors with static factory methods for Sum.[*]Fn classes

• Type: Improvement
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 0.5.0
• Component/s: sdk-java-core
• Labels: Done backward-incompatible
• Assignee: Stas Levin
• Reporter: Stas Levin
• Created: 28/Dec/16 08:07
• Updated: 06/Oct/20 21:59
• Resolved: 17/Jan/17 09:45

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.

Issue Links

• links to: GitHub Pull Request #1723
• links to: GitHub Pull Request #1731