diff --git a/core/trino-main/src/test/java/io/trino/sql/planner/assertions/JoinMatcher.java b/core/trino-main/src/test/java/io/trino/sql/planner/assertions/JoinMatcher.java index b6c790b55dd8..6b8f4ee2bd26 100644 --- a/core/trino-main/src/test/java/io/trino/sql/planner/assertions/JoinMatcher.java +++ b/core/trino-main/src/test/java/io/trino/sql/planner/assertions/JoinMatcher.java @@ -196,7 +196,7 @@ public String toString() public static class Builder { - private JoinNode.Type joinType; + private final JoinNode.Type joinType; private Optional>> equiCriteria = Optional.empty(); private Optional> dynamicFilter = Optional.empty(); private Optional distributionType = Optional.empty(); @@ -205,11 +205,9 @@ public static class Builder private PlanMatchPattern right; private Optional filter = Optional.empty(); - public Builder type(JoinNode.Type joinType) + public Builder(JoinNode.Type joinType) { this.joinType = joinType; - - return this; } public Builder equiCriteria(List> expectedEquiCriteria) diff --git a/core/trino-main/src/test/java/io/trino/sql/planner/assertions/PlanMatchPattern.java b/core/trino-main/src/test/java/io/trino/sql/planner/assertions/PlanMatchPattern.java index 7315f4d21a14..fa0aa1fe90ac 100644 --- a/core/trino-main/src/test/java/io/trino/sql/planner/assertions/PlanMatchPattern.java +++ b/core/trino-main/src/test/java/io/trino/sql/planner/assertions/PlanMatchPattern.java @@ -415,7 +415,7 @@ public static PlanMatchPattern patternRecognition(Consumer handler) { - JoinMatcher.Builder builder = new JoinMatcher.Builder().type(type); + JoinMatcher.Builder builder = new JoinMatcher.Builder(type); handler.accept(builder); return builder.build(); }