[ 
https://issues.apache.org/jira/browse/BEAM-1169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17283462#comment-17283462
 ] 

Brian Hulette commented on BEAM-1169:
-------------------------------------

Was it? There's still a comment in the code pointing to this jira: 
https://github.com/apache/beam/blob/9ebdc8afcfd316333299c6929ad67ead8cf1ae4e/sdks/java/core/src/test/java/org/apache/beam/sdk/metrics/MetricsTest.java#L348

Also the test that links it, testAllAttemptedMetrics, is flaky (BEAM-7827), 
which seems related

> MetricsTest matchers should loosen expectations on attempted values
> -------------------------------------------------------------------
>
>                 Key: BEAM-1169
>                 URL: https://issues.apache.org/jira/browse/BEAM-1169
>             Project: Beam
>          Issue Type: Sub-task
>          Components: beam-model, sdk-java-core, sdk-py-core
>            Reporter: Ben Chambers
>            Priority: P3
>              Labels: Clarified, Done
>
> We could use `atLeast(N)` rather than `equals(N)` for the attempted values, 
> but even that may be false without violating the behavior.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to