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 <[email protected]>
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);
}