[#1958] improvement(common): replace stream.peek with map (#2104)

### What changes were proposed in this pull request? replace stream.peek with map ### Why are the changes needed? Fix: #1958 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? By hand Co-authored-by: YongXing <[email protected]>
apache · Feb 6, 2024 · c9d9787 · c9d9787
1 parent 4275d21
commit c9d9787
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/common/src/main/java/com/datastrato/gravitino/dto/requests/TableCreateRequest.java b/common/src/main/java/com/datastrato/gravitino/dto/requests/TableCreateRequest.java
@@ -127,7 +127,11 @@ public void validate() throws IllegalArgumentException {
 
     List<ColumnDTO> autoIncrementCols =
         Arrays.stream(columns)
-            .peek(ColumnDTO::validate)
+            .map(
+                column -> {
+                  column.validate();
+                  return column;
+                })
             .filter(ColumnDTO::autoIncrement)
             .collect(Collectors.toList());
     String autoIncrementColsStr =