[ https://issues.apache.org/jira/browse/FLINK-7797?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16311793#comment-16311793 ]
ASF GitHub Bot commented on FLINK-7797: --------------------------------------- Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5140#discussion_r159695882 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/join/TimeBoundedStreamJoin.scala --- @@ -196,35 +181,38 @@ abstract class TimeBoundedStreamJoin( if (rightTime >= rightQualifiedLowerBound && rightTime <= rightQualifiedUpperBound) { val rightRows = rightEntry.getValue var i = 0 - var markEmitted = false + var entryUpdated = false while (i < rightRows.size) { - joinCollector.resetThisTurn() + joinCollector.reset() val tuple = rightRows.get(i) joinFunction.join(leftRow, tuple.f0, joinCollector) - if (joinType == JoinType.RIGHT_OUTER || joinType == JoinType.FULL_OUTER) { - if (!tuple.f1 && joinCollector.everEmittedThisTurn) { - // Mark the right row as being successfully joined and emitted. - tuple.f1 = true - markEmitted = true + if (joinCollector.emitted) { --- End diff -- change to ``` emitted = emitted || joinCollector.emitted if (joinType == JoinType.RIGHT_OUTER || joinType == JoinType.FULL_OUTER) { if (!tuple.f1 && joinCollector.emitted) { // Mark the right row as being successfully joined and emitted. tuple.f1 = true entryUpdated = true } } ``` to avoid a condition for inner and left joins > Add support for windowed outer joins for streaming tables > --------------------------------------------------------- > > Key: FLINK-7797 > URL: https://issues.apache.org/jira/browse/FLINK-7797 > Project: Flink > Issue Type: Sub-task > Components: Table API & SQL > Affects Versions: 1.4.0 > Reporter: Fabian Hueske > Assignee: Xingcan Cui > > Currently, only windowed inner joins for streaming tables are supported. > This issue is about adding support for windowed LEFT, RIGHT, and FULL OUTER > joins. -- This message was sent by Atlassian JIRA (v6.4.14#64029)