[
https://issues.apache.org/jira/browse/FLINK-2531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14698578#comment-14698578
]
ASF GitHub Bot commented on FLINK-2531:
---------------------------------------
GitHub user Rucongzhang opened a pull request:
https://github.com/apache/flink/pull/1023
[FLINK-2531]combining the if branch to improve the judge logic
In function receiveAcknowledgeMessage, the "if" branch
"checkpoint.isFullyAcknowledged()" can combine with the "if " branch "if
(checkpoint != null && !checkpoint.isDiscarded()) " to improve judge logic.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Rucongzhang/flink FLINK-2531
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1023.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 #1023
----
commit ab2eddd0ff43c97b472265c5dc7b8bd541fd76ed
Author: Rucongzhang <[email protected]>
Date: 2015-08-16T07:56:38Z
[FLINK-2531]combining the if branch to improve the judge logic
----
> combining the "if" branch to improve the performance
> ----------------------------------------------------
>
> Key: FLINK-2531
> URL: https://issues.apache.org/jira/browse/FLINK-2531
> Project: Flink
> Issue Type: Bug
> Reporter: zhangrucong
> Priority: Minor
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
