[
https://issues.apache.org/jira/browse/BEAM-8869?focusedWorklogId=353207&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-353207
]
ASF GitHub Bot logged work on BEAM-8869:
----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Dec/19 02:51
Start Date: 04/Dec/19 02:51
Worklog Time Spent: 10m
Work Description: kennknowles commented on pull request #10266:
[BEAM-8869] Exclude system metrics test from legacy runner test suite
URL: https://github.com/apache/beam/pull/10266#discussion_r353527004
##########
File path:
runners/core-java/src/test/java/org/apache/beam/runners/core/metrics/MetricsPusherTest.java
##########
@@ -73,7 +74,12 @@ public void pushesUserMetrics() throws Exception {
assertThat(TestMetricsSink.getCounterValue(COUNTER_NAME),
is(NUM_ELEMENTS));
}
- @Category({ValidatesRunner.class, UsesAttemptedMetrics.class,
UsesCounterMetrics.class})
+ @Category({
+ ValidatesRunner.class,
+ UsesAttemptedMetrics.class,
+ UsesCounterMetrics.class,
+ UsesSystemMetrics.class
Review comment:
Just to double check since I don't really know what all the categories mean
- is it possible that `UsesMetricsPusher` is relevant?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 353207)
Time Spent: 1h 10m (was: 1h)
> MetricsPusherTest.pushesSystemMetrics is failing on non-portability
> FlinkRunner
> -------------------------------------------------------------------------------
>
> Key: BEAM-8869
> URL: https://issues.apache.org/jira/browse/BEAM-8869
> Project: Beam
> Issue Type: Bug
> Components: runner-flink, test-failures
> Reporter: Kenneth Knowles
> Assignee: Michal Walenia
> Priority: Critical
> Labels: currently-failing
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> Started right when it was merged:
> [https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/5971/]
> Recent:
> [https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/5986/]
> {code:java}
> java.lang.AssertionError:
> Expected: is <false>
> but: was <true>
> at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
> at org.junit.Assert.assertThat(Assert.java:966)
> at org.junit.Assert.assertThat(Assert.java:931)
> at
> org.apache.beam.runners.core.metrics.MetricsPusherTest.pushesSystemMetrics(MetricsPusherTest.java:89)
> {code}
> The assertion is:
> {code:java}
> assertThat(TestMetricsSink.getSystemCounters().isEmpty(), is(false));
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
