Apply suggestions from code review

Co-authored-by: Scott Gerring <[email protected]>
aws-powertools · Dec 6, 2023 · 75f866f · 75f866f
1 parent 5ba6d1a
commit 75f866f
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/powertools-logging/pom.xml b/powertools-logging/pom.xml
@@ -27,7 +27,7 @@
     <packaging>jar</packaging>
 
     <name>Powertools for AWS Lambda (Java) library Logging</name>
-    <description>Set of utility for better logging - common</description>
+    <description>Set of utilities to improve Lambda function handler logging - common</description>
     <url>https://aws.amazon.com/lambda/</url>
     <issueManagement>
         <system>GitHub Issues</system>

diff --git a/...ng-logback/src/main/java/software/amazon/lambda/powertools/logging/LambdaJsonEncoder.java b/...ng-logback/src/main/java/software/amazon/lambda/powertools/logging/LambdaJsonEncoder.java
@@ -28,7 +28,6 @@
 /**
  * Custom encoder for logback that encodes logs in JSON format.
  * It does not use a JSON library but a custom serializer ({@link LambdaJsonSerializer})
- * to reduce the weight of the library.
  */
 public class LambdaJsonEncoder extends EncoderBase<ILoggingEvent> {
 

diff --git a/...rc/main/java/software/amazon/lambda/powertools/logging/internal/LambdaJsonSerializer.java b/...rc/main/java/software/amazon/lambda/powertools/logging/internal/LambdaJsonSerializer.java
@@ -34,8 +34,7 @@
 /**
  * This class will serialize the log events in json.<br/>
  * <p>
- * Inspired from the ElasticSearch Serializer co.elastic.logging.EcsJsonSerializer, this class doesn't use
- * any JSON (de)serialization library (Jackson, Gson, etc.) to avoid the dependency
+ * Inspired from the ElasticSearch Serializer co.elastic.logging.EcsJsonSerializer
  */
 public class LambdaJsonSerializer {
     protected static final String TIMESTAMP_ATTR_NAME = "timestamp";