lint

Author: xinsong-cui

tests/benchmarks/service-benchmarks/jvm/src/aws/sdk/kotlin/benchmarks/service/Common.kt

@@ -23,11 +23,9 @@ object Common {
 
     fun random(prefix: String = "") = "$prefix${Uuid.random()}"
 
-    fun randomStringPayload(scale: Int): String {
-        return (1..scale)
-            .map { Random.nextInt(32, 127).toChar() }
-            .joinToString("")
-    }
+    fun randomStringPayload(scale: Int): String = (1..scale)
+        .map { Random.nextInt(32, 127).toChar() }
+        .joinToString("")
 
     fun randomBytesPayload(scale: Int): ByteArray {
         val bytes = ByteArray(scale)

tests/benchmarks/service-benchmarks/jvm/src/aws/sdk/kotlin/benchmarks/service/ProtocolBenchmarkHarness.kt

@@ -1,3 +1,7 @@
+/*
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
 package aws.sdk.kotlin.benchmarks.service
 
 import aws.sdk.kotlin.benchmarks.service.definitions.*
@@ -49,15 +53,14 @@ class ProtocolBenchmarkHarness {
         operation.setup(client)
 
         try {
-            if(operation.requireScaling){
-                for(scale in scales){
-                    println("    Warming up for ${operation.warmupMode.explanation} (scale: ${scale})...")
+            if (operation.requireScaling) {
+                for (scale in scales) {
+                    println("    Warming up for ${operation.warmupMode.explanation} (scale: $scale)...")
                     forAtLeast(operation.warmupMode) { iteration ->
                         operation.transact(client, scale, iteration)
                     }
                 }
-            }
-            else{
+            } else {
                 println("    Warming up for ${operation.warmupMode.explanation}...")
                 forAtLeast(operation.warmupMode) { iteration ->
                     operation.transact(client, 0, iteration)
@@ -66,15 +69,14 @@ class ProtocolBenchmarkHarness {
 
             Common.metricAggregator.clear()
 
-            if(operation.requireScaling) {
+            if (operation.requireScaling) {
                 for (scale in scales) {
-                    println("    Measuring for ${operation.iterationMode.explanation} (scale: ${scale})...")
+                    println("    Measuring for ${operation.iterationMode.explanation} (scale: $scale)...")
                     forAtLeast(operation.iterationMode) { iteration ->
                         operation.transact(client, scale, iteration)
                     }
                 }
-            }
-            else{
+            } else {
                 println("    Measuring for ${operation.iterationMode.explanation}...")
                 forAtLeast(operation.iterationMode) { iteration ->
                     operation.transact(client, 0, iteration)

tests/benchmarks/service-benchmarks/jvm/src/aws/sdk/kotlin/benchmarks/service/definitions/CloudwatchProtocolBenchmark.kt

@@ -1,3 +1,7 @@
+/*
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
 package aws.sdk.kotlin.benchmarks.service.definitions
 
 import aws.sdk.kotlin.benchmarks.service.Common
@@ -7,8 +11,8 @@ import aws.smithy.kotlin.runtime.ExperimentalApi
 import aws.smithy.kotlin.runtime.time.Instant
 import aws.smithy.kotlin.runtime.time.fromEpochMilliseconds
 import java.util.*
-import kotlin.time.Duration.Companion.seconds
 import kotlin.random.Random
+import kotlin.time.Duration.Companion.seconds
 
 class CloudwatchProtocolBenchmark : ServiceProtocolBenchmark<CloudWatchClient> {
     companion object {
@@ -32,7 +36,7 @@ class CloudwatchProtocolBenchmark : ServiceProtocolBenchmark<CloudWatchClient> {
     override val operations get() = listOf(
         putMetricDataBenchmark,
         getMetricDataBenchmark,
-        listMetricsBenchmark
+        listMetricsBenchmark,
     )
 
     private val putMetricDataBenchmark = object : AbstractOperationProtocolBenchmark<CloudWatchClient>("Put metric data") {
@@ -47,8 +51,8 @@ class CloudwatchProtocolBenchmark : ServiceProtocolBenchmark<CloudWatchClient> {
                         dimensions = listOf(
                             Dimension {
                                 name = "TestDimension"
-                                value = "$suiteId-${scale}"
-                            }
+                                value = "$suiteId-$scale"
+                            },
                         )
                         value = Random.nextDouble()
                         unit = null
@@ -83,13 +87,13 @@ class CloudwatchProtocolBenchmark : ServiceProtocolBenchmark<CloudWatchClient> {
                                 dimensions = listOf(
                                     Dimension {
                                         name = "TestDimension"
-                                        value = "$suiteId-${scale}"
-                                    }
+                                        value = "$suiteId-$scale"
+                                    },
                                 )
                             }
                             period = 60
                         }
-                    }
+                    },
                 )
             }
 
@@ -103,13 +107,13 @@ class CloudwatchProtocolBenchmark : ServiceProtocolBenchmark<CloudWatchClient> {
         override val requireScaling = true
 
         override suspend fun transact(client: CloudWatchClient, scale: Int, iteration: Int) {
-            client.listMetrics (
+            client.listMetrics(
                 ListMetricsRequest {
                     namespace = "TestNamespace"
-                }
+                },
             )
 
             if ((iteration % 50) == 0) Thread.sleep(2000)
         }
     }
-}
+}

tests/benchmarks/service-benchmarks/jvm/src/aws/sdk/kotlin/benchmarks/service/definitions/SecretsManagerProtocolBenchmark.kt

@@ -37,7 +37,7 @@ class SecretsManagerProtocolBenchmark : ServiceProtocolBenchmark<SecretsManagerC
                 Tag {
                     key = "Iteration"
                     value = "${iteration.padded()}"
-                }
+                },
             )
 
             client.createSecret {
@@ -157,7 +157,7 @@ class SecretsManagerProtocolBenchmark : ServiceProtocolBenchmark<SecretsManagerC
                 Filter {
                     key = FilterNameStringType.fromValue("tag-value")
                     values = listOf("${iteration.padded()}")
-                }
+                },
             )
 
             client.listSecrets {

tests/benchmarks/service-benchmarks/jvm/src/aws/sdk/kotlin/benchmarks/service/definitions/ServiceProtocolBenchmark.kt

@@ -1,7 +1,11 @@
+/*
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
 package aws.sdk.kotlin.benchmarks.service.definitions
 
-import aws.sdk.kotlin.benchmarks.service.DEFAULT_WARMUP_ITERATIONS
 import aws.sdk.kotlin.benchmarks.service.DEFAULT_ITERATIONS
+import aws.sdk.kotlin.benchmarks.service.DEFAULT_WARMUP_ITERATIONS
 import aws.smithy.kotlin.runtime.client.SdkClient
 
 /**
@@ -39,6 +43,7 @@ interface OperationProtocolBenchmark<C : SdkClient> : OperationBenchmark<C> {
      * `RunMode.Iterations(DEFAULT_ITERATIONS)`.
      */
     override val iterationMode: RunMode get() = RunMode.Iterations(DEFAULT_ITERATIONS)
+
     /**
      * Extended transact method with scale and iteration parameters.
      * @param client The service client to use
@@ -59,5 +64,5 @@ interface OperationProtocolBenchmark<C : SdkClient> : OperationBenchmark<C> {
  */
 abstract class AbstractOperationProtocolBenchmark<C : SdkClient>(
     override val name: String,
-    override val requireScaling: Boolean = false
-) : OperationProtocolBenchmark<C>
+    override val requireScaling: Boolean = false,
+) : OperationProtocolBenchmark<C>