GitHub user peihe opened a pull request:
https://github.com/apache/beam/pull/3235
Backport 2
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
- [ ] Make sure the PR title is formatted like:
`[BEAM-<Jira issue #>] Description of pull request`
- [ ] Make sure tests pass via `mvn clean verify`.
- [ ] Replace `<Jira issue #>` in the title with the actual Jira issue
number, if there is one.
- [ ] If this contribution is large, please file an Apache
[Individual Contributor License
Agreement](https://www.apache.org/licenses/icla.pdf).
---
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/peihe/incubator-beam backport-2
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/beam/pull/3235.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3235
----
commit c991ee140662b83ed5ac5a825840bfff4206ddf0
Author: Author: æ³¢ç¹ <[email protected]>
Date: 2017-05-26T09:40:27Z
ReduceFnRunner.onTrigger: skip storeCurrentPaneInfo() if trigger isFinished.
commit f49fa854751e5a2816d5ff3d639bea8b0c742186
Author: Author: æ³¢ç¹ <[email protected]>
Date: 2017-05-26T09:42:46Z
ReduceFnRunner.onTrigger: add short circuit for FIRE_IF_NON_EMPTY.
commit 8dab4ca73dbf3a066a5e1d9d10df08a98415814f
Author: Author: æ³¢ç¹ <[email protected]>
Date: 2017-05-26T09:46:55Z
ReduceFnRunner.onTrigger: move inputWM and pane after the short circuit,
since they may not be needed.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---