TACO-199 - remove unneeded comments

xjdr · Aug 6, 2018 · 4ee5a36 · 4ee5a36
1 parent 918da86
commit 4ee5a36
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 19 deletions.
diff --git a/xio-core/src/main/java/com/xjeffrose/xio/tracing/XioTracing.java b/xio-core/src/main/java/com/xjeffrose/xio/tracing/XioTracing.java
@@ -29,7 +29,7 @@ Tracing buildTracing(@NonNull String name, @NonNull String zipkinUrl, float samp
         // puts trace IDs into logs
         .currentTraceContext(MDCCurrentTraceContext.create())
         .spanReporter(buildReporter(zipkinUrl))
-        .sampler(Sampler.create(samplingRate)) // TODO: Investigate BoundarySampler
+        .sampler(Sampler.create(samplingRate))
         .build();
   }
 

diff --git a/...core/src/test/java/com/xjeffrose/xio/tracing/HttpClientTracingHandlerIntegrationTest.java b/...core/src/test/java/com/xjeffrose/xio/tracing/HttpClientTracingHandlerIntegrationTest.java
@@ -141,22 +141,4 @@ public void testOutBoundAndInboundSpan() throws Exception {
     byte[] bytes = Hex.decodeHex(responseHex.toCharArray());
     assertEquals(expectedResponse, new String(bytes, "UTF-8"));
   }
-
-  //  // @Override
-  //  @Test(expected = ComparisonFailure.class)
-  //  public void redirect() throws Exception {
-  //    throw new AssumptionViolatedException("client does not support redirect");
-  //  }
-  //
-  //  // @Override
-  //  @Test(expected = ComparisonFailure.class)
-  //  public void addsStatusCodeWhenNotOk() throws Exception {
-  //    throw new AssumptionViolatedException("test is flaky");
-  //  }
-  //
-  //  // @Override
-  //  @Test(expected = ComparisonFailure.class)
-  //  public void httpPathTagExcludesQueryParams() throws Exception {
-  //    throw new AssumptionViolatedException("test is flaky");
-  //  }
 }