fixed checkstyle and used StringBuilder

Author: roamariei

services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/extensions/AutoGeneratedTimestampRecordExtension.java

@@ -153,9 +153,14 @@ public WriteModification beforeWrite(DynamoDbExtensionContext.BeforeWrite contex
             } else if (value.hasL() && !value.l().isEmpty() && value.l().get(0).hasM()) {
                 TableSchema<?> elementListSchema = getTableSchemaForListElement(context.tableSchema(), key);
 
-                List<AttributeValue> updatedList = value.l().stream()
+                List<AttributeValue> updatedList = value.l()
+                                                        .stream()
                                                         .map(listItem -> listItem.hasM() ?
-                                                                         AttributeValue.builder().m(processNestedObject(listItem.m(), elementListSchema, currentInstant)).build() : listItem)
+                                                                         AttributeValue.builder()
+                                                                                       .m(processNestedObject(listItem.m(),
+                                                                                                              elementListSchema,
+                                                                                                              currentInstant))
+                                                                                       .build() : listItem)
                                                         .collect(Collectors.toList());
                 updatedItems.put(key, AttributeValue.builder().l(updatedList).build());
             }
@@ -165,7 +170,8 @@ public WriteModification beforeWrite(DynamoDbExtensionContext.BeforeWrite contex
 
         stringTableSchemaMap.forEach((path, schema) -> {
             Collection<String> customMetadataObject = schema.tableMetadata()
-                                                            .customMetadataObject(CUSTOM_METADATA_KEY, Collection.class).orElse(null);
+                                                            .customMetadataObject(CUSTOM_METADATA_KEY, Collection.class)
+                                                            .orElse(null);
 
             if (customMetadataObject != null) {
                 customMetadataObject.forEach(
@@ -203,7 +209,8 @@ private TableSchema<?> getTableSchemaForListElement(TableSchema<?> rootSchema, S
                 }
                 String attributeName = parts[parts.length - 1];
                 listElementSchema = TableSchema.fromClass(
-                    Class.forName(currentSchema.converterForAttribute(attributeName).type().rawClassParameters().get(0).rawClass().getName()));
+                    Class.forName(currentSchema.converterForAttribute(attributeName)
+                                               .type().rawClassParameters().get(0).rawClass().getName()));
             }
         } catch (ClassNotFoundException e) {
             throw new IllegalArgumentException("Class not found for field name: " + key, e);
@@ -219,11 +226,16 @@ private Map<String, TableSchema<?>> resolveSchemasPerPath(Map<String, AttributeV
         for (String key : attributesToSet.keySet()) {
             String[] parts = NESTED_OBJECT_PATTERN.split(key);
 
-            String path = "";
+            StringBuilder pathBuilder = new StringBuilder();
             TableSchema<?> currentSchema = rootSchema;
 
             for (int i = 0; i < parts.length - 1; i++) {
-                path = path.isEmpty() ? parts[i] : path + "." + parts[i];
+                if (pathBuilder.length() > 0) {
+                    pathBuilder.append(".");
+                }
+                pathBuilder.append(parts[i]);
+
+                String path = pathBuilder.toString();
 
                 if (!schemaMap.containsKey(path)) {
                     Optional<? extends TableSchema<?>> nestedSchema = getNestedSchema(currentSchema, parts[i]);
@@ -253,7 +265,7 @@ private Map<String, AttributeValue> processNestedObject(Map<String, AttributeVal
         Collection<String> customMetadataObject = nestedSchema.tableMetadata()
                                                       .customMetadataObject(CUSTOM_METADATA_KEY, Collection.class).orElse(null);
 
-        if (customMetadataObject!= null) {
+        if (customMetadataObject != null) {
             customMetadataObject.forEach(
                 key -> insertTimestampInItemToTransform(updatedNestedMap, String.valueOf(key),
                                                         nestedSchema.converterForAttribute(key), currentInstant));
@@ -268,11 +280,17 @@ private Map<String, AttributeValue> processNestedObject(Map<String, AttributeVal
                        && nestedValue.l().get(0).hasM()) {
                 try {
                     TableSchema<?> listElementSchema = TableSchema.fromClass(
-                        Class.forName(nestedSchema.converterForAttribute(nestedKey).type().rawClassParameters().get(0).rawClass().getName()));
-                    List<AttributeValue> updatedList = nestedValue.l().stream()
-                                                                  .map(listItem -> listItem.hasM() ?
-                                                                                   AttributeValue.builder().m(processNestedObject(listItem.m(), listElementSchema, currentInstant)).build() : listItem)
-                                                                  .collect(Collectors.toList());
+                        Class.forName(nestedSchema.converterForAttribute(nestedKey)
+                                                  .type().rawClassParameters().get(0).rawClass().getName()));
+                    List<AttributeValue> updatedList = nestedValue
+                        .l()
+                        .stream()
+                        .map(listItem -> listItem.hasM() ?
+                                         AttributeValue.builder()
+                                                       .m(processNestedObject(listItem.m(),
+                                                                              listElementSchema,
+                                                                              currentInstant)).build() : listItem)
+                        .collect(Collectors.toList());
                     updatedNestedMap.put(nestedKey, AttributeValue.builder().l(updatedList).build());
                 } catch (ClassNotFoundException e) {
                     throw new IllegalArgumentException("Class not found for field name: " + nestedKey, e);

services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/internal/update/UpdateExpressionUtils.java

@@ -90,16 +90,17 @@ private static List<SetAction> setActionsFor(Map<String, AttributeValue> attribu
             if (key.contains(NESTED_OBJECT_UPDATE)) {
                 TableSchema currentSchema = tableSchema;
                 List<String> pathFieldNames = Arrays.asList(PATTERN.split(key));
-                String attributeName = pathFieldNames.get(pathFieldNames.size()-1);
+                String attributeName = pathFieldNames.get(pathFieldNames.size() - 1);
 
-                for(int i = 0; i < pathFieldNames.size() - 1; i++ ) {
+                for (int i = 0; i < pathFieldNames.size() - 1; i++) {
                     Optional<? extends TableSchema<?>> nestedSchema = getNestedSchema(currentSchema, pathFieldNames.get(i));
                     if (nestedSchema.isPresent()) {
                         currentSchema = nestedSchema.get();
                     }
                 }
 
-                actions.add(setValue(key, value, UpdateBehaviorTag.resolveForAttribute(attributeName, currentSchema.tableMetadata())));
+                actions.add(setValue(key, value,
+                                     UpdateBehaviorTag.resolveForAttribute(attributeName, currentSchema.tableMetadata())));
             } else {
                 actions.add(setValue(key, value, UpdateBehaviorTag.resolveForAttribute(key, tableSchema.tableMetadata())));
             }