From b0839d2440be30f67ac216b556adb6565a475bd3 Mon Sep 17 00:00:00 2001 From: wusheng Date: Sat, 3 Dec 2016 18:54:31 +0800 Subject: [PATCH] 1. solve compile issue. 2. remove test-api module --- .../com/a/eye/skywalking/network/Server.java | 5 +- skywalking-sniffer/pom.xml | 1 - .../skywalking-test-api/pom.xml | 29 ---- .../api/ContextPoolOperator.java | 69 --------- .../testframework/api/RequestSpanAssert.java | 137 ------------------ .../testframework/api/config/Config.java | 12 -- .../exception/SpanDataFormatException.java | 7 - .../exception/SpanDataNotEqualsException.java | 7 - .../exception/TraceIdNotSameException.java | 7 - .../TraceNodeSizeNotEqualException.java | 7 - .../skywalking/reflect/SubClassReflect.java | 25 ---- .../com/ai/skywalking/reflect/TestClass.java | 10 -- 12 files changed, 2 insertions(+), 314 deletions(-) delete mode 100644 skywalking-sniffer/skywalking-test-api/pom.xml delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/ContextPoolOperator.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/RequestSpanAssert.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/config/Config.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataFormatException.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataNotEqualsException.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceIdNotSameException.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceNodeSizeNotEqualException.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/SubClassReflect.java delete mode 100644 skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/TestClass.java diff --git a/skywalking-network/src/main/java/com/a/eye/skywalking/network/Server.java b/skywalking-network/src/main/java/com/a/eye/skywalking/network/Server.java index b558d77bc..e674d2bd7 100644 --- a/skywalking-network/src/main/java/com/a/eye/skywalking/network/Server.java +++ b/skywalking-network/src/main/java/com/a/eye/skywalking/network/Server.java @@ -3,7 +3,6 @@ package com.a.eye.skywalking.network; import com.a.eye.skywalking.network.grpc.server.AsyncTraceSearchServer; import com.a.eye.skywalking.network.grpc.server.SpanStorageServer; import com.a.eye.skywalking.network.grpc.server.TraceSearchServer; -import com.a.eye.skywalking.network.listener.server.AsyncTraceSearchServerListener; import com.a.eye.skywalking.network.listener.server.SpanStorageServerListener; import com.a.eye.skywalking.network.listener.server.TraceSearchListener; import io.grpc.netty.NettyServerBuilder; @@ -61,8 +60,8 @@ public class Server { return this; } - public TransferServiceBuilder addAsyncTraceSearchService(AsyncTraceSearchServerListener asyncTraceSearchServerListener){ - serverBuilder.addService(new AsyncTraceSearchServer(asyncTraceSearchServerListener)); + public TransferServiceBuilder addAsyncTraceSearchService(TraceSearchListener traceSearchListener){ + serverBuilder.addService(new AsyncTraceSearchServer(traceSearchListener)); return this; } } diff --git a/skywalking-sniffer/pom.xml b/skywalking-sniffer/pom.xml index 885bf00c4..0a2db3b37 100644 --- a/skywalking-sniffer/pom.xml +++ b/skywalking-sniffer/pom.xml @@ -16,7 +16,6 @@ skywalking-agent skywalking-api skywalking-sdk-plugin - skywalking-test-api diff --git a/skywalking-sniffer/skywalking-test-api/pom.xml b/skywalking-sniffer/skywalking-test-api/pom.xml deleted file mode 100644 index f9d8e968d..000000000 --- a/skywalking-sniffer/skywalking-test-api/pom.xml +++ /dev/null @@ -1,29 +0,0 @@ - - 4.0.0 - - - com.a.eye - skywalking-sniffer - 2.0-2016 - - - skywalking-test-api - jar - - skywalking-test-api - http://maven.apache.org - - - UTF-8 - - - - - com.a.eye - skywalking-network - ${project.version} - - - - diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/ContextPoolOperator.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/ContextPoolOperator.java deleted file mode 100644 index 7b5b83454..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/ContextPoolOperator.java +++ /dev/null @@ -1,69 +0,0 @@ -package com.a.eye.skywalking.testframework.api; - -import com.a.eye.skywalking.testframework.api.config.Config; - -import java.lang.reflect.Field; -import java.util.ArrayList; -import java.util.List; - -public class ContextPoolOperator { - public static List acquireBufferData() { - List resultSpan = new ArrayList(); - Object[] bufferGroupObjectArray = acquireBufferGroupObjectArrayByClassLoader(); - - for (Object bufferGroup : bufferGroupObjectArray) { - Object[] spanList = acquireBufferData(bufferGroup); - for (Object span : spanList) { - if (span != null) { - resultSpan.add(span); - } - } - } - - return resultSpan; - } - - public static void clearSpanData() { - Object[] bufferGroupObjectArray = acquireBufferGroupObjectArrayByClassLoader(); - - for (Object bufferGroup : bufferGroupObjectArray) { - Object[] spanList = acquireBufferData(bufferGroup); - for (int i = 0; i < spanList.length; i++) { - spanList[i] = null; - } - } - } - - private static Object[] acquireBufferData(Object bufferGroup) { - try { - Class bufferGroupClass = Thread.currentThread().getContextClassLoader().loadClass(Config.BUFFER_GROUP_CLASS_NAME); - Field spanArrayField = bufferGroupClass.getDeclaredField(Config.SPAN_ARRAY_FIELD_NAME); - spanArrayField.setAccessible(true); - return (Object[]) spanArrayField.get(bufferGroup); - } catch (Exception e) { - throw new RuntimeException("Failed to acquire span array", e); - } - } - - - private static Object[] acquireBufferGroupObjectArrayByClassLoader() { - try { - Class bufferPoolClass = fetchBufferPoolClass(); - Field field = fetchBufferPoolObject(bufferPoolClass); - return (Object[]) field.get(bufferPoolClass); - } catch (Exception e) { - throw new RuntimeException("Failed to acquire data group object array", e); - } - } - - private static Field fetchBufferPoolObject(Class bufferPoolClass) throws NoSuchFieldException { - Field field = bufferPoolClass.getDeclaredField(Config.BUFFER_GROUP_FIELD_NAME); - field.setAccessible(true); - return field; - } - - private static Class fetchBufferPoolClass() throws ClassNotFoundException { - ClassLoader classLoader = Thread.currentThread().getContextClassLoader(); - return classLoader.loadClass(Config.BUFFER_POOL_CLASS_NAME); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/RequestSpanAssert.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/RequestSpanAssert.java deleted file mode 100644 index 29840355f..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/RequestSpanAssert.java +++ /dev/null @@ -1,137 +0,0 @@ -package com.a.eye.skywalking.testframework.api; - -import com.a.eye.skywalking.network.grpc.RequestSpan; -import com.a.eye.skywalking.network.grpc.TraceId; -import com.a.eye.skywalking.testframework.api.exception.SpanDataNotEqualsException; -import com.a.eye.skywalking.testframework.api.exception.SpanDataFormatException; -import com.a.eye.skywalking.testframework.api.exception.TraceIdNotSameException; -import com.a.eye.skywalking.testframework.api.exception.TraceNodeSizeNotEqualException; - -import java.util.ArrayList; -import java.util.List; - -public class RequestSpanAssert { - - public static void assertEquals(String[][] expectedRequestSpan) { - assertEquals(expectedRequestSpan, false); - } - - public static void assertEquals(String[][] expectedRequestSpan, boolean skipValidateTraceId) { - List requestSpan = acquiredRequestSpanFromBuffer(); - - if (!skipValidateTraceId) { - validateTraceId(requestSpan); - } - - List assertSpanData = convertSpanDataToCompareStr(requestSpan); - - List expectedSpanData = convertSpanDataToCompareStr(expectedRequestSpan); - - validateTraceSpanSize(expectedSpanData.size(), assertSpanData.size()); - - validateSpanData(expectedSpanData, assertSpanData); - - } - - private static List acquiredRequestSpanFromBuffer() { - List spans = ContextPoolOperator.acquireBufferData(); - - List result = new ArrayList(); - for (Object span : spans) { - if (span instanceof RequestSpan) { - result.add((RequestSpan) span); - } - } - return result; - } - - public static void clearTraceData() { - ContextPoolOperator.clearSpanData(); - } - - private static List convertSpanDataToCompareStr(List assertSpanData) { - List resultSpanData = new ArrayList(); - for (RequestSpan span : assertSpanData) { - StringBuffer tmpSpanDataStr = new StringBuffer(jointTraceLevelId(span.getParentLevel(), span.getLevelId() + " ")); - tmpSpanDataStr.append(span.getViewPointId().trim() + " "); - tmpSpanDataStr.append(span.getBusinessKey() == null ? " " : span.getBusinessKey() + " "); - - resultSpanData.add(tmpSpanDataStr.toString().trim()); - } - return resultSpanData; - } - - private static String jointTraceLevelId(String parentLevelId, String levelId) { - String traceLevelId = ""; - if (parentLevelId != null && parentLevelId.length() > 0) { - traceLevelId = parentLevelId + "."; - } - traceLevelId += levelId; - return traceLevelId; - } - - private static List convertSpanDataToCompareStr(String[][] assertTraceTree) { - List resultSpanData = new ArrayList(); - for (String[] spanDataArray : assertTraceTree) { - if (spanDataArray.length != 3) { - throw new SpanDataFormatException("assert trace tree is illegal, " + "Format :\ttraceLevelId\t|\tviewPoint\t|\tbusinesskey"); - } - - StringBuffer tmpSpanDataStr = new StringBuffer(spanDataArray[0] + " "); - tmpSpanDataStr.append(spanDataArray[1] == null ? " " : spanDataArray[1].trim() + " ").append(spanDataArray[2] == null ? " " : spanDataArray[2].trim() + " "); - - resultSpanData.add(tmpSpanDataStr.toString().trim()); - } - - return resultSpanData; - } - - - private static void validateTraceSpanSize(int expectedSpanSize, int actualSpanSize) { - if (actualSpanSize != expectedSpanSize) { - throw new TraceNodeSizeNotEqualException("expected span size : " + expectedSpanSize + - "\n actual span size : " + actualSpanSize); - } - } - - private static void validateSpanData(List expectedSpanData, List assertTraceTree) { - for (String assertSpanDataStr : assertTraceTree) { - if (expectedSpanData.contains(assertSpanDataStr)) { - expectedSpanData.remove(assertSpanDataStr); - } - } - - if (expectedSpanData.size() != 0) { - StringBuffer stringBuffer = new StringBuffer(); - for (String expectedSpan : expectedSpanData) { - stringBuffer.append(expectedSpan + "\n"); - } - - throw new SpanDataNotEqualsException("actual trace tree is not contain those span as follow:\n" + stringBuffer); - } - - } - - private static void validateTraceId(List traceSpanList) { - String traceId = null; - for (RequestSpan span : traceSpanList) { - if (traceId == null) { - traceId = toLiteralLTraceId(span.getTraceId()); - } - - if (!traceId.equals(toLiteralLTraceId(span.getTraceId()))) { - throw new TraceIdNotSameException("trace id is not all the same.trace id :" + - traceId + ",Error trace id :" + span.getTraceId()); - } - - } - } - - private static String toLiteralLTraceId(TraceId traceId){ - StringBuilder tid = new StringBuilder(); - for (Long segment : traceId.getSegmentsList()) { - tid.append(segment).append("."); - } - return tid.substring(0, tid.length() - 1); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/config/Config.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/config/Config.java deleted file mode 100644 index c20cb2170..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/config/Config.java +++ /dev/null @@ -1,12 +0,0 @@ -package com.a.eye.skywalking.testframework.api.config; - -public class Config { - - public static String BUFFER_POOL_CLASS_NAME = "com.a.eye.skywalking.data.BufferPool"; - - public static String BUFFER_GROUP_FIELD_NAME = "bufferGroups"; - - public static String BUFFER_GROUP_CLASS_NAME = "com.a.eye.skywalking.data.BufferGroup"; - - public static String SPAN_ARRAY_FIELD_NAME = "dataBuffer"; -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataFormatException.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataFormatException.java deleted file mode 100644 index 5630fc33e..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataFormatException.java +++ /dev/null @@ -1,7 +0,0 @@ -package com.a.eye.skywalking.testframework.api.exception; - -public class SpanDataFormatException extends RuntimeException { - public SpanDataFormatException(String message) { - super(message); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataNotEqualsException.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataNotEqualsException.java deleted file mode 100644 index 7b2252ac4..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/SpanDataNotEqualsException.java +++ /dev/null @@ -1,7 +0,0 @@ -package com.a.eye.skywalking.testframework.api.exception; - -public class SpanDataNotEqualsException extends RuntimeException { - public SpanDataNotEqualsException(String message) { - super(message); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceIdNotSameException.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceIdNotSameException.java deleted file mode 100644 index a171baac1..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceIdNotSameException.java +++ /dev/null @@ -1,7 +0,0 @@ -package com.a.eye.skywalking.testframework.api.exception; - -public class TraceIdNotSameException extends RuntimeException { - public TraceIdNotSameException(String message) { - super(message); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceNodeSizeNotEqualException.java b/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceNodeSizeNotEqualException.java deleted file mode 100644 index 322653d11..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/main/java/com/a/eye/skywalking/testframework/api/exception/TraceNodeSizeNotEqualException.java +++ /dev/null @@ -1,7 +0,0 @@ -package com.a.eye.skywalking.testframework.api.exception; - -public class TraceNodeSizeNotEqualException extends RuntimeException { - public TraceNodeSizeNotEqualException(String message) { - super(message); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/SubClassReflect.java b/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/SubClassReflect.java deleted file mode 100644 index c1bfd7ca1..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/SubClassReflect.java +++ /dev/null @@ -1,25 +0,0 @@ -package test.com.ai.skywalking.reflect; - -import org.junit.Test; - -import java.lang.reflect.Field; - -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; - -/** - * Created by xin on 16-6-6. - */ -public class SubClassReflect { - - @Test - public void fetchSubClassField() throws ClassNotFoundException, NoSuchFieldException, IllegalAccessException { - ClassLoader loader = Thread.currentThread().getContextClassLoader(); - Class testSubClass = loader.loadClass("test.com.ai.skywalking.reflect.TestClass$TestSubClass"); - Field field = testSubClass.getDeclaredField("testStringArray"); - assertNotNull(field); - field.setAccessible(true); - Object[] objects = (Object[]) field.get(testSubClass); - assertEquals(5, objects.length); - } -} diff --git a/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/TestClass.java b/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/TestClass.java deleted file mode 100644 index 2f3a73f60..000000000 --- a/skywalking-sniffer/skywalking-test-api/src/test/java/test/com/ai/skywalking/reflect/TestClass.java +++ /dev/null @@ -1,10 +0,0 @@ -package test.com.ai.skywalking.reflect; - -public class TestClass { - - private static TestSubClass testSubClass = new TestSubClass(); - - static class TestSubClass { - private static String[] testStringArray = new String[5]; - } -} -- GitLab