Remove unnecessary conversion methods

prestodb · May 29, 2024 · 88bc1c3 · 88bc1c3
1 parent 88e5c77
commit 88bc1c3
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/sql/planner/optimizations/KeyBasedSampler.java b/presto-main/src/main/java/com/facebook/presto/sql/planner/optimizations/KeyBasedSampler.java
@@ -178,7 +178,7 @@ private PlanNode addSamplingFilter(PlanNode tableScanNode, Optional<VariableRefe
                 tableName = ((TableScanNode) tableScanNode).getTable().getConnectorHandle().toString();
             }
             else {
-                tableName = "plan node: " + String.valueOf(tableScanNode.getId());
+                tableName = "plan node: " + tableScanNode.getId();
             }
 
             sampledFields.add(String.format("%s from %s", rowExpression, tableName));

diff --git a/presto-redis/src/main/java/com/facebook/presto/redis/RedisRecordCursor.java b/presto-redis/src/main/java/com/facebook/presto/redis/RedisRecordCursor.java
@@ -279,9 +279,9 @@ private ScanParams setScanParams()
             if (redisJedisManager.getRedisConnectorConfig().isKeyPrefixSchemaTable()) {
                 String keyMatch = "";
                 if (!split.getSchemaName().equals("default")) {
-                    keyMatch = split.getSchemaName() + Character.toString(redisJedisManager.getRedisConnectorConfig().getRedisKeyDelimiter());
+                    keyMatch = split.getSchemaName() + redisJedisManager.getRedisConnectorConfig().getRedisKeyDelimiter();
                 }
-                keyMatch = keyMatch + split.getTableName() + Character.toString(redisJedisManager.getRedisConnectorConfig().getRedisKeyDelimiter()) + "*";
+                keyMatch = keyMatch + split.getTableName() + redisJedisManager.getRedisConnectorConfig().getRedisKeyDelimiter() + "*";
                 scanParams.match(keyMatch);
             }
             return scanParams;

diff --git a/presto-redis/src/test/java/com/facebook/presto/redis/util/CodecSupplier.java b/presto-redis/src/test/java/com/facebook/presto/redis/util/CodecSupplier.java
@@ -63,7 +63,7 @@ protected Type _deserialize(String value, DeserializationContext context)
         {
             Type type = metadata.getType(parseTypeSignature(value));
             if (type == null) {
-                throw new IllegalArgumentException(String.valueOf("Unknown type " + value));
+                throw new IllegalArgumentException("Unknown type " + value);
             }
             return type;
         }

diff --git a/presto-tests/src/main/java/com/facebook/presto/tests/DistributedQueryRunner.java b/presto-tests/src/main/java/com/facebook/presto/tests/DistributedQueryRunner.java
@@ -811,7 +811,7 @@ private void enableTestFunctionNamespaces(List<String> catalogNames, Map<String,
     {
         checkState(testFunctionNamespacesHandle.get() == null, "Test function namespaces already enabled");
 
-        String databaseName = String.valueOf(nanoTime()) + "_" + ThreadLocalRandom.current().nextInt();
+        String databaseName = nanoTime() + "_" + ThreadLocalRandom.current().nextInt();
         Map<String, String> properties = ImmutableMap.<String, String>builder()
                 .put("database-name", databaseName)
                 .putAll(additionalProperties)