Rename java packages for lettuce 4 and 5

DataDog · May 12, 2020 · cc2d949 · cc2d949
1 parent c20caa9
commit cc2d949
Show file tree

Hide file tree

Showing 23 changed files with 46 additions and 48 deletions.
diff --git a/.../LettuceAsyncCommandsInstrumentation.java → .../LettuceAsyncCommandsInstrumentation.java b/.../LettuceAsyncCommandsInstrumentation.java → .../LettuceAsyncCommandsInstrumentation.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
 import static java.util.Collections.singletonMap;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;

diff --git a/...lettuce/LettuceClientInstrumentation.java → ...ettuce4/LettuceClientInstrumentation.java b/...lettuce/LettuceClientInstrumentation.java → ...ettuce4/LettuceClientInstrumentation.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
 import static java.util.Collections.singletonMap;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;

diff --git a/...tation/lettuce/InstrumentationPoints.java → ...ation/lettuce4/InstrumentationPoints.java b/...tation/lettuce/InstrumentationPoints.java → ...ation/lettuce4/InstrumentationPoints.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
 import static com.lambdaworks.redis.protocol.CommandKeyword.SEGFAULT;
 import static com.lambdaworks.redis.protocol.CommandType.CLIENT;

diff --git a/...n/lettuce/LettuceAsyncCommandsAdvice.java → .../lettuce4/LettuceAsyncCommandsAdvice.java b/...n/lettuce/LettuceAsyncCommandsAdvice.java → .../lettuce4/LettuceAsyncCommandsAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
 import com.lambdaworks.redis.protocol.AsyncCommand;
 import com.lambdaworks.redis.protocol.RedisCommand;
@@ -20,5 +20,4 @@ public static void onExit(
       @Advice.Return final AsyncCommand<?, ?, ?> asyncCommand) {
     InstrumentationPoints.afterCommand(command, scope, throwable, asyncCommand);
   }
-
 }
diff --git a/...ation/lettuce/LettuceClientDecorator.java → ...tion/lettuce4/LettuceClientDecorator.java b/...ation/lettuce/LettuceClientDecorator.java → ...tion/lettuce4/LettuceClientDecorator.java
@@ -1,6 +1,6 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
-import static datadog.trace.instrumentation.lettuce.InstrumentationPoints.getCommandResourceName;
+import static datadog.trace.instrumentation.lettuce4.InstrumentationPoints.getCommandResourceName;
 
 import com.lambdaworks.redis.RedisURI;
 import com.lambdaworks.redis.protocol.RedisCommand;
@@ -40,17 +40,17 @@ protected String dbType() {
   }
 
   @Override
-  protected String dbUser(RedisURI connection) {
+  protected String dbUser(final RedisURI connection) {
     return null;
   }
 
   @Override
-  protected String dbInstance(RedisURI connection) {
+  protected String dbInstance(final RedisURI connection) {
     return null;
   }
 
   @Override
-  public AgentSpan onConnection(AgentSpan span, RedisURI connection) {
+  public AgentSpan onConnection(final AgentSpan span, final RedisURI connection) {
     if (connection != null) {
       span.setTag(Tags.PEER_HOSTNAME, connection.getHost());
       span.setTag(Tags.PEER_PORT, connection.getPort());
@@ -60,18 +60,19 @@ public AgentSpan onConnection(AgentSpan span, RedisURI connection) {
     return super.onConnection(span, connection);
   }
 
-  public AgentSpan onCommand(AgentSpan span, RedisCommand<?, ?, ?> command) {
-    span.setTag(DDTags.RESOURCE_NAME,
+  public AgentSpan onCommand(final AgentSpan span, final RedisCommand<?, ?, ?> command) {
+    span.setTag(
+        DDTags.RESOURCE_NAME,
         null == command ? "Redis Command" : getCommandResourceName(command.getType()));
     return span;
   }
 
-  private static String resourceName(RedisURI connection) {
+  private static String resourceName(final RedisURI connection) {
     return "CONNECT:"
-      + connection.getHost()
-      + ":"
-      + connection.getPort()
-      + "/"
-      + connection.getDatabase();
+        + connection.getHost()
+        + ":"
+        + connection.getPort()
+        + "/"
+        + connection.getDatabase();
   }
 }
diff --git a/...tation/lettuce/RedisConnectionAdvice.java → ...ation/lettuce4/RedisConnectionAdvice.java b/...tation/lettuce/RedisConnectionAdvice.java → ...ation/lettuce4/RedisConnectionAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce4;
 
 import com.lambdaworks.redis.RedisURI;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
@@ -12,8 +12,8 @@ public static AgentScope onEnter(@Advice.Argument(1) final RedisURI redisURI) {
   }
 
   @Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class)
-  public static void onExit(@Advice.Enter final AgentScope scope,
-                            @Advice.Thrown final Throwable throwable) {
+  public static void onExit(
+      @Advice.Enter final AgentScope scope, @Advice.Thrown final Throwable throwable) {
     InstrumentationPoints.afterConnect(scope, throwable);
   }
 }
diff --git a/...test/groovy/LettuceAsyncClientTest.groovy → ...est/groovy/Lettuce4AsyncClientTest.groovy b/...test/groovy/LettuceAsyncClientTest.groovy → ...est/groovy/Lettuce4AsyncClientTest.groovy
@@ -1,20 +1,20 @@
 import com.lambdaworks.redis.ClientOptions
 import com.lambdaworks.redis.RedisClient
+import com.lambdaworks.redis.RedisConnectionException
 import com.lambdaworks.redis.RedisFuture
 import com.lambdaworks.redis.RedisURI
 import com.lambdaworks.redis.api.StatefulConnection
 import com.lambdaworks.redis.api.async.RedisAsyncCommands
 import com.lambdaworks.redis.api.sync.RedisCommands
+import com.lambdaworks.redis.codec.Utf8StringCodec
 import com.lambdaworks.redis.protocol.AsyncCommand
 import datadog.trace.agent.test.AgentTestRunner
 import datadog.trace.agent.test.utils.PortUtils
 import datadog.trace.api.DDSpanTypes
 import datadog.trace.bootstrap.instrumentation.api.Tags
-
 import redis.embedded.RedisServer
 import spock.lang.Shared
 import spock.util.concurrent.AsyncConditions
-import com.lambdaworks.redis.codec.Utf8StringCodec
 
 import java.util.concurrent.CancellationException
 import java.util.concurrent.TimeUnit
@@ -23,11 +23,9 @@ import java.util.function.BiFunction
 import java.util.function.Consumer
 import java.util.function.Function
 
-import com.lambdaworks.redis.RedisConnectionException
-
-import static datadog.trace.instrumentation.lettuce.InstrumentationPoints.AGENT_CRASHING_COMMAND_PREFIX
+import static datadog.trace.instrumentation.lettuce4.InstrumentationPoints.AGENT_CRASHING_COMMAND_PREFIX
 
-class LettuceAsyncClientTest extends AgentTestRunner {
+class Lettuce4AsyncClientTest extends AgentTestRunner {
   public static final String HOST = "127.0.0.1"
   public static final int DB_INDEX = 0
   // Disable autoreconnect so we do not get stray traces popping up on server shutdown
@@ -107,7 +105,7 @@ class LettuceAsyncClientTest extends AgentTestRunner {
 
     when:
     StatefulConnection connection = testConnectionClient.connect(new Utf8StringCodec(),
-        new RedisURI(HOST, port, 3, TimeUnit.SECONDS))
+      new RedisURI(HOST, port, 3, TimeUnit.SECONDS))
 
     then:
     connection != null

diff --git a/.../test/groovy/LettuceSyncClientTest.groovy → ...test/groovy/Lettuce4SyncClientTest.groovy b/.../test/groovy/LettuceSyncClientTest.groovy → ...test/groovy/Lettuce4SyncClientTest.groovy
@@ -10,9 +10,9 @@ import datadog.trace.bootstrap.instrumentation.api.Tags
 import redis.embedded.RedisServer
 import spock.lang.Shared
 
-import static datadog.trace.instrumentation.lettuce.InstrumentationPoints.AGENT_CRASHING_COMMAND_PREFIX
+import static datadog.trace.instrumentation.lettuce4.InstrumentationPoints.AGENT_CRASHING_COMMAND_PREFIX
 
-class LettuceSyncClientTest extends AgentTestRunner {
+class Lettuce4SyncClientTest extends AgentTestRunner {
   public static final String HOST = "127.0.0.1"
   public static final int DB_INDEX = 0
   // Disable autoreconnect so we do not get stray traces popping up on server shutdown

diff --git a/.../LettuceAsyncCommandsInstrumentation.java → .../LettuceAsyncCommandsInstrumentation.java b/.../LettuceAsyncCommandsInstrumentation.java → .../LettuceAsyncCommandsInstrumentation.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static java.util.Collections.singletonMap;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;

diff --git a/...lettuce/LettuceClientInstrumentation.java → ...ettuce5/LettuceClientInstrumentation.java b/...lettuce/LettuceClientInstrumentation.java → ...ettuce5/LettuceClientInstrumentation.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static java.util.Collections.singletonMap;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;

diff --git a/...ttuceReactiveCommandsInstrumentation.java → ...ttuceReactiveCommandsInstrumentation.java b/...ttuceReactiveCommandsInstrumentation.java → ...ttuceReactiveCommandsInstrumentation.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
 import static net.bytebuddy.matcher.ElementMatchers.isPublic;

diff --git a/...ation/lettuce/ConnectionFutureAdvice.java → ...tion/lettuce5/ConnectionFutureAdvice.java b/...ation/lettuce/ConnectionFutureAdvice.java → ...tion/lettuce5/ConnectionFutureAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.startSpan;

diff --git a/...ation/lettuce/LettuceAsyncBiFunction.java → ...tion/lettuce5/LettuceAsyncBiFunction.java b/...ation/lettuce/LettuceAsyncBiFunction.java → ...tion/lettuce5/LettuceAsyncBiFunction.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static datadog.trace.instrumentation.lettuce.LettuceClientDecorator.DECORATE;
 

diff --git a/...n/lettuce/LettuceAsyncCommandsAdvice.java → .../lettuce5/LettuceAsyncCommandsAdvice.java b/...n/lettuce/LettuceAsyncCommandsAdvice.java → .../lettuce5/LettuceAsyncCommandsAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.startSpan;

diff --git a/...ation/lettuce/LettuceClientDecorator.java → ...tion/lettuce5/LettuceClientDecorator.java b/...ation/lettuce/LettuceClientDecorator.java → ...tion/lettuce5/LettuceClientDecorator.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import datadog.trace.api.DDSpanTypes;
 import datadog.trace.api.DDTags;

diff --git a/...n/lettuce/LettuceInstrumentationUtil.java → .../lettuce5/LettuceInstrumentationUtil.java b/...n/lettuce/LettuceInstrumentationUtil.java → .../lettuce5/LettuceInstrumentationUtil.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce;
+package datadog.trace.instrumentation.lettuce5;
 
 import io.lettuce.core.protocol.RedisCommand;
 import java.util.Arrays;

diff --git a/...lettuce/rx/LettuceFluxCreationAdvice.java → ...ettuce5/rx/LettuceFluxCreationAdvice.java b/...lettuce/rx/LettuceFluxCreationAdvice.java → ...ettuce5/rx/LettuceFluxCreationAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce.rx;
+package datadog.trace.instrumentation.lettuce5.rx;
 
 import static datadog.trace.instrumentation.lettuce.LettuceInstrumentationUtil.expectsResponse;
 

diff --git a/...ce/rx/LettuceFluxTerminationRunnable.java → ...e5/rx/LettuceFluxTerminationRunnable.java b/...ce/rx/LettuceFluxTerminationRunnable.java → ...e5/rx/LettuceFluxTerminationRunnable.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce.rx;
+package datadog.trace.instrumentation.lettuce5.rx;
 
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.startSpan;
 import static datadog.trace.instrumentation.lettuce.LettuceClientDecorator.DECORATE;

diff --git a/...lettuce/rx/LettuceMonoCreationAdvice.java → ...ettuce5/rx/LettuceMonoCreationAdvice.java b/...lettuce/rx/LettuceMonoCreationAdvice.java → ...ettuce5/rx/LettuceMonoCreationAdvice.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce.rx;
+package datadog.trace.instrumentation.lettuce5.rx;
 
 import static datadog.trace.instrumentation.lettuce.LettuceInstrumentationUtil.expectsResponse;
 

diff --git a/...n/lettuce/rx/LettuceMonoDualConsumer.java → .../lettuce5/rx/LettuceMonoDualConsumer.java b/...n/lettuce/rx/LettuceMonoDualConsumer.java → .../lettuce5/rx/LettuceMonoDualConsumer.java
@@ -1,4 +1,4 @@
-package datadog.trace.instrumentation.lettuce.rx;
+package datadog.trace.instrumentation.lettuce5.rx;
 
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.startSpan;
 import static datadog.trace.instrumentation.lettuce.LettuceClientDecorator.DECORATE;

diff --git a/...test/groovy/LettuceAsyncClientTest.groovy → ...est/groovy/Lettuce5AsyncClientTest.groovy b/...test/groovy/LettuceAsyncClientTest.groovy → ...est/groovy/Lettuce5AsyncClientTest.groovy
@@ -25,9 +25,9 @@ import java.util.function.BiFunction
 import java.util.function.Consumer
 import java.util.function.Function
 
-import static datadog.trace.instrumentation.lettuce.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
+import static datadog.trace.instrumentation.lettuce5.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
 
-class LettuceAsyncClientTest extends AgentTestRunner {
+class Lettuce5AsyncClientTest extends AgentTestRunner {
   public static final String HOST = "127.0.0.1"
   public static final int DB_INDEX = 0
   // Disable autoreconnect so we do not get stray traces popping up on server shutdown

diff --git a/...t/groovy/LettuceReactiveClientTest.groovy → .../groovy/Lettuce5ReactiveClientTest.groovy b/...t/groovy/LettuceReactiveClientTest.groovy → .../groovy/Lettuce5ReactiveClientTest.groovy
@@ -18,9 +18,9 @@ import spock.util.concurrent.AsyncConditions
 import java.util.function.Consumer
 
 import static datadog.trace.agent.test.utils.TraceUtils.runUnderTrace
-import static datadog.trace.instrumentation.lettuce.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
+import static datadog.trace.instrumentation.lettuce5.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
 
-class LettuceReactiveClientTest extends AgentTestRunner {
+class Lettuce5ReactiveClientTest extends AgentTestRunner {
   public static final String HOST = "127.0.0.1"
   public static final int DB_INDEX = 0
   // Disable autoreconnect so we do not get stray traces popping up on server shutdown

diff --git a/.../test/groovy/LettuceSyncClientTest.groovy → ...test/groovy/Lettuce5SyncClientTest.groovy b/.../test/groovy/LettuceSyncClientTest.groovy → ...test/groovy/Lettuce5SyncClientTest.groovy
@@ -12,9 +12,9 @@ import spock.lang.Shared
 
 import java.util.concurrent.CompletionException
 
-import static datadog.trace.instrumentation.lettuce.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
+import static datadog.trace.instrumentation.lettuce5.LettuceInstrumentationUtil.AGENT_CRASHING_COMMAND_PREFIX
 
-class LettuceSyncClientTest extends AgentTestRunner {
+class Lettuce5SyncClientTest extends AgentTestRunner {
   public static final String HOST = "127.0.0.1"
   public static final int DB_INDEX = 0
   // Disable autoreconnect so we do not get stray traces popping up on server shutdown