You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by ki...@apache.org on 2022/01/19 15:18:01 UTC

[incubator-seatunnel] branch dev updated: [Improve] Delete useless variable and simplify lambda for CheckConfigUtil (#1110)

This is an automated email from the ASF dual-hosted git repository.

kirs pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/incubator-seatunnel.git


The following commit(s) were added to refs/heads/dev by this push:
     new f5265db  [Improve] Delete useless variable and simplify lambda for CheckConfigUtil (#1110)
f5265db is described below

commit f5265dbdc47be699d0d4e9ec3d470ecbda887489
Author: Benedict Jin <as...@apache.org>
AuthorDate: Wed Jan 19 23:17:53 2022 +0800

    [Improve] Delete useless variable and simplify lambda for CheckConfigUtil (#1110)
---
 .../main/java/org/apache/seatunnel/common/config/CheckConfigUtil.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/seatunnel-common/src/main/java/org/apache/seatunnel/common/config/CheckConfigUtil.java b/seatunnel-common/src/main/java/org/apache/seatunnel/common/config/CheckConfigUtil.java
index b810213..f00cede 100644
--- a/seatunnel-common/src/main/java/org/apache/seatunnel/common/config/CheckConfigUtil.java
+++ b/seatunnel-common/src/main/java/org/apache/seatunnel/common/config/CheckConfigUtil.java
@@ -73,12 +73,11 @@ public class CheckConfigUtil {
      * merge all check result
      */
     public static CheckResult mergeCheckMessage(CheckResult... checkResults) {
-        List<String> list = new LinkedList<>();
         List<CheckResult> notPassConfig = Arrays.stream(checkResults).filter(item -> !item.isSuccess()).collect(Collectors.toList());
         if (notPassConfig.isEmpty()) {
             return new CheckResult(true, CHECK_SUCCESS);
         } else {
-            String errMessage = notPassConfig.stream().map(it -> it.getMsg()).collect(Collectors.joining(","));
+            String errMessage = notPassConfig.stream().map(CheckResult::getMsg).collect(Collectors.joining(","));
             return new CheckResult(false, errMessage);
         }