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

jark pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new bcc994b  [hotfix][table-runtime-blink] Fix checkstyle of 
NestedLoopJoinCodeGenerator
bcc994b is described below

commit bcc994be7d9e5b5ba80ee4362770bdb38f9134f5
Author: Jark Wu <imj...@gmail.com>
AuthorDate: Fri Jul 5 10:26:01 2019 +0800

    [hotfix][table-runtime-blink] Fix checkstyle of NestedLoopJoinCodeGenerator
---
 .../flink/table/codegen/NestedLoopJoinCodeGenerator.scala   | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git 
a/flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/codegen/NestedLoopJoinCodeGenerator.scala
 
b/flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/codegen/NestedLoopJoinCodeGenerator.scala
index 264a52a..b909899 100644
--- 
a/flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/codegen/NestedLoopJoinCodeGenerator.scala
+++ 
b/flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/codegen/NestedLoopJoinCodeGenerator.scala
@@ -180,17 +180,18 @@ class NestedLoopJoinCodeGenerator(
         ctx.addReusableMember(s"$bitSetTerm $buildMatched = null;")
       }
     }
+    val collectorRow = if (leftIsBuild) {
+      joinedRow(buildNullRow, probeRow)
+    } else {
+      joinedRow(probeRow, buildNullRow)
+    }
 
     val probeOuterCode =
       s"""
          |if (!$probeMatched) {
-         |  ${generateCollect(
-              if (leftIsBuild)
-                joinedRow(buildNullRow, probeRow)
-              else
-                joinedRow(probeRow, buildNullRow))}
+         |  ${generateCollect(collectorRow)}
          |}
-            """.stripMargin
+       """.stripMargin
 
     val iterCnt = newName("iteratorCount")
     val joinBuildAndProbe = {

Reply via email to