[#noissue] Remove unused api

- Remove TraceContext#currentRpcTraceObject API
likeonechild · Jun 5, 2017 · 3bec16f · 3bec16f
1 parent b4b2465
commit 3bec16f
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 53 deletions.
diff --git a/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/DefaultTraceContext.java b/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/DefaultTraceContext.java
@@ -119,10 +119,6 @@ public Trace currentTraceObject() {
         return traceFactory.currentTraceObject();
     }
 
-    public Trace currentRpcTraceObject() {
-        return traceFactory.currentTraceObject();
-    }
-
     /**
      * Return trace without sampling check.
      *

diff --git a/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/ThreadLocalTraceFactory.java b/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/ThreadLocalTraceFactory.java
@@ -63,20 +63,6 @@ public Trace currentTraceObject() {
         return null;
     }
 
-    /**
-     * Return Trace object without validating
-     *
-     * @return
-     */
-    @Override
-    public Trace currentRpcTraceObject() {
-        final Trace trace = threadLocalBinder.get();
-        if (trace == null) {
-            return null;
-        }
-        return trace;
-    }
-
     @Override
     public Trace currentRawTraceObject() {
         return threadLocalBinder.get();

diff --git a/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/TraceFactory.java b/profiler/src/main/java/com/navercorp/pinpoint/profiler/context/TraceFactory.java
@@ -25,8 +25,6 @@ public interface TraceFactory extends BaseTraceFactory {
 
     Trace currentTraceObject();
 
-    Trace currentRpcTraceObject();
-
     Trace currentRawTraceObject();
 
     Trace removeTraceObject();

diff --git a/...iler/src/main/java/com/navercorp/pinpoint/profiler/context/active/ActiveTraceFactory.java b/...iler/src/main/java/com/navercorp/pinpoint/profiler/context/active/ActiveTraceFactory.java
@@ -52,11 +52,6 @@ public Trace currentTraceObject() {
         return this.delegate.currentTraceObject();
     }
 
-    @Override
-    public Trace currentRpcTraceObject() {
-        return this.delegate.currentRpcTraceObject();
-    }
-
     @Override
     public Trace currentRawTraceObject() {
         return this.delegate.currentRawTraceObject();

diff --git a/...er/src/test/java/com/navercorp/pinpoint/profiler/context/ThreadLocalTraceFactoryTest.java b/...er/src/test/java/com/navercorp/pinpoint/profiler/context/ThreadLocalTraceFactoryTest.java
@@ -65,9 +65,6 @@ public void nullTraceObject() {
         Trace rawTraceObject = traceFactory.currentRawTraceObject();
         Assert.assertNull(rawTraceObject);
 
-        Trace rpcTraceObject = traceFactory.currentRpcTraceObject();
-        Assert.assertNull(rpcTraceObject);
-
         traceFactory.newTraceObject();
         Assert.assertNotNull(traceFactory.currentRawTraceObject());
     }
@@ -96,27 +93,7 @@ public void testCurrentTraceObject_unsampled() throws Exception {
         Assert.assertNotEquals(newTrace, currentTrace);
     }
 
-    @Test
-    public void testCurrentRpcTraceObject() throws Exception {
-        TraceFactory traceFactory = sampledTraceFactory;
-
-        Trace trace = traceFactory.newTraceObject();
-        Trace rpcTraceObject = traceFactory.currentRpcTraceObject();
-
-        Assert.assertNotNull(rpcTraceObject);
-        Assert.assertSame(trace, rpcTraceObject);
-    }
 
-    @Test
-    public void testCurrentRpcTraceObject_unsampled() throws Exception {
-        TraceFactory traceFactory = unsampledTraceFactory;
-
-        Trace trace = traceFactory.newTraceObject();
-        Trace rpcTraceObject = traceFactory.currentRpcTraceObject();
-
-        Assert.assertNotNull(rpcTraceObject);
-        Assert.assertSame(trace, rpcTraceObject);
-    }
 
     @Test
     public void testCurrentRawTraceObject() throws Exception {

diff --git a/.../src/test/java/com/navercorp/pinpoint/profiler/monitor/metric/rpc/MetricTraceFactory.java b/.../src/test/java/com/navercorp/pinpoint/profiler/monitor/metric/rpc/MetricTraceFactory.java
@@ -47,11 +47,6 @@ public Trace currentTraceObject() {
         return delegate.currentTraceObject();
     }
 
-    @Override
-    public Trace currentRpcTraceObject() {
-        return delegate.currentRpcTraceObject();
-    }
-
     @Override
     public Trace currentRawTraceObject() {
         return delegate.currentRawTraceObject();