Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Remove unnecessary gradle definition #183

Merged
merged 1 commit into from May 15, 2017

Conversation

pavolloffay
Copy link
Member

No description provided.

@yurishkuro
Copy link
Member

Looks like a flaky test (#184)

com.uber.jaeger.reporters.RemoteReporterTest > testFlushUpdatesQueueLength FAILED
    java.lang.AssertionError: expected:<0> but was:<1>
        at org.junit.Assert.fail(Assert.java:88)
        at org.junit.Assert.failNotEquals(Assert.java:834)
        at org.junit.Assert.assertEquals(Assert.java:645)
        at org.junit.Assert.assertEquals(Assert.java:631)
        at com.uber.jaeger.reporters.RemoteReporterTest.testFlushUpdatesQueueLength(RemoteReporterTest.java:213)

@codecov-io
Copy link

Codecov Report

Merging #183 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #183      +/-   ##
============================================
- Coverage     80.94%   80.89%   -0.06%     
+ Complexity      461      460       -1     
============================================
  Files            77       77              
  Lines          1790     1790              
  Branches        210      210              
============================================
- Hits           1449     1448       -1     
  Misses          251      251              
- Partials         90       91       +1
Impacted Files Coverage Δ Complexity Δ
.../uber/jaeger/samplers/RemoteControlledSampler.java 91.54% <0%> (-1.41%) 18% <0%> (-1%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f955e48...77e24eb. Read the comment docs.

@yurishkuro yurishkuro merged commit d94c1da into jaegertracing:master May 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants