[Backport] Reduce CPU cycles spent on setting tracing tags (#3339) (#3504)

(cherry picked from commit 5d96c67)

Co-authored-by: Rohan Nagar <[email protected]>
Co-authored-by: Tihomir Mateev <[email protected]>

Author: 3 people

src/main/java/io/lettuce/core/protocol/CommandHandler.java

@@ -32,17 +32,13 @@
 import java.util.Collection;
 import java.util.LinkedHashSet;
 import java.util.List;
-import java.util.Optional;
 import java.util.Queue;
 import java.util.Set;
 import java.util.concurrent.atomic.AtomicLong;
 
 import io.lettuce.core.ClientOptions;
-import io.lettuce.core.ConnectionBuilder;
 import io.lettuce.core.RedisConnectionException;
-import io.lettuce.core.RedisCredentials;
 import io.lettuce.core.RedisException;
-import io.lettuce.core.RedisURI;
 import io.lettuce.core.api.push.PushListener;
 import io.lettuce.core.api.push.PushMessage;
 import io.lettuce.core.datastructure.queue.HashIndexedQueue;
@@ -491,15 +487,6 @@ private void attachTracing(ChannelHandlerContext ctx, RedisCommand<?, ?, ?> comm
             Tracer.Span span = tracer.nextSpan(context);
             span.name(command.getType().toString());
 
-            if (channel.hasAttr(ConnectionBuilder.REDIS_URI)) {
-                String redisUriStr = channel.attr(ConnectionBuilder.REDIS_URI).get();
-                RedisURI redisURI = RedisURI.create(redisUriStr);
-                span.tag("server.address", redisURI.toString());
-                span.tag("db.namespace", String.valueOf(redisURI.getDatabase()));
-                span.tag("user.name", Optional.ofNullable(redisURI.getCredentialsProvider().resolveCredentials().block())
-                        .map(RedisCredentials::getUsername).orElse(""));
-            }
-
             if (tracedEndpoint != null) {
                 span.remoteEndpoint(tracedEndpoint);
             } else {

src/test/java/io/lettuce/core/tracing/SynchronousIntegrationTests.java

@@ -78,9 +78,6 @@ public SampleTestRunnerConsumer yourCode() {
                         .containsEntry("net.sock.peer.addr", TestSettings.host())
                         .containsEntry("net.sock.peer.port", "" + TestSettings.port());
                 assertThat(finishedSpan.getTags()).containsKeys("db.operation");
-                assertThat(finishedSpan.getTags()).containsKeys("server.address");
-                assertThat(finishedSpan.getTags()).containsKeys("db.namespace");
-                assertThat(finishedSpan.getTags()).containsKeys("user.name");
             }
 
             assertThat(commands).extracting(RedisCommand::getType).contains(CommandType.PING, CommandType.HELLO);