[
https://issues.apache.org/jira/browse/BEAM-3527?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kenneth Knowles resolved BEAM-3527.
-----------------------------------
Resolution: Fixed
Fix Version/s: 2.4.0
Fixed by doing a rename and doc to make it clear what is going on.
> org.apache.beam.sdk.metrics.DistributionResult mixes up min and max
> constructor args
> -------------------------------------------------------------------------------------
>
> Key: BEAM-3527
> URL: https://issues.apache.org/jira/browse/BEAM-3527
> Project: Beam
> Issue Type: Bug
> Components: runner-core
> Reporter: Claire McGinty
> Assignee: Kenneth Knowles
> Priority: Minor
> Fix For: 2.4.0
>
> Time Spent: 50m
> Remaining Estimate: 0h
>
> [GitHub link to
> class|https://github.com/apache/beam/blob/a39960b131674e6c1d120a971b91b37226c88414/sdks/java/core/src/main/java/org/apache/beam/sdk/metrics/DistributionResult.java]
> given the constructor:
>
> {code:java}
> public static DistributionResult create(long sum, long count, long min, long
> max){code}
>
> the ZERO value
>
> {code:java}
> public static final DistributionResult ZERO = create(0, 0, Long.MAX_VALUE,
> Long.MIN_VALUE){code}
>
> should be:
>
> {code:java}
> public static final DistributionResult ZERO = create(0, 0, Long.MIN_VALUE,
> Long.MAX_VALUE);{code}
>
> unless this is intentional...?
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
