ktlint

Author: lauzadis

runtime/observability/logging-crt/native/src/aws/smithy/kotlin/runtime/telemetry/logging/crt/CrtLogRecordBuilder.kt

@@ -15,7 +15,6 @@ public class CrtLogRecordBuilder(
 ) : LogRecordBuilder {
     private var cause: Throwable? = null
     private var msg: (() -> String)? = null
-    private val kvp by lazy { mutableMapOf<String, Any>() }
 
     override fun setCause(ex: Throwable) {
         cause = ex
@@ -29,9 +28,8 @@ public class CrtLogRecordBuilder(
         msg = message
     }
 
-    override fun setKeyValuePair(key: String, value: Any) {
-        kvp[key] = value
-    }
+    // CRT logger does not support setting key-value pairs
+    override fun setKeyValuePair(key: String, value: Any) { }
 
     override fun emit() {
         val message = requireNotNull(msg) { "no message provided to LogRecordBuilder" }

runtime/observability/logging-crt/native/src/aws/smithy/kotlin/runtime/telemetry/logging/crt/CrtLogger.kt

@@ -14,7 +14,9 @@ import aws.smithy.kotlin.runtime.telemetry.logging.MessageSupplier
 import aws.sdk.kotlin.crt.Config as CrtConfig
 import aws.sdk.kotlin.crt.LogLevel as CrtLogLevel
 
-public class CrtLogger(public val name: String, public val config: CrtConfig) : Logger, WithCrt() {
+public class CrtLogger(public val name: String, public val config: CrtConfig) :
+    WithCrt(),
+    Logger {
     override fun trace(t: Throwable?, msg: MessageSupplier) {
         log(CrtLogLevel.Trace, msg())
     }

runtime/observability/logging-crt/native/src/aws/smithy/kotlin/runtime/telemetry/logging/crt/CrtLoggerProvider.kt

@@ -5,11 +5,10 @@
 
 package aws.smithy.kotlin.runtime.telemetry.logging.crt
 
-import aws.sdk.kotlin.crt.Config as CrtConfig
 import aws.smithy.kotlin.runtime.telemetry.logging.*
+import aws.sdk.kotlin.crt.Config as CrtConfig
 
 public class CrtLoggerProvider : LoggerProvider {
     override fun getOrCreateLogger(name: String): Logger = CrtLogger(name, CrtConfig())
     public fun getOrCreateLogger(name: String, config: CrtConfig): Logger = CrtLogger(name, config)
 }
-