You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by Ethanlm <gi...@git.apache.org> on 2017/10/16 15:14:23 UTC

[GitHub] storm pull request #2372: [STORM-2777] The number of ackers should default t...

GitHub user Ethanlm opened a pull request:

    https://github.com/apache/storm/pull/2372

    [STORM-2777] The number of ackers should default to the number of workers

    JIRA: https://issues.apache.org/jira/browse/STORM-2777
    
    https://github.com/apache/storm/pull/2325 introduced a bug that  the number of ackers doesn't default to the number of workers in non-RAS cases.
    
    Manually tested on both non-RAS case and RAS case.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Ethanlm/storm STORM-2777

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/2372.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 #2372
    
----
commit 767c7f3be8699099ad9b2ffbdfc8baab36dd14f0
Author: Ethan Li <et...@gmail.com>
Date:   2017-10-16T15:11:42Z

    [STORM-2777] The number of ackers should default to the number of workers

----


---

[GitHub] storm pull request #2372: [STORM-2777] The number of ackers should default t...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/2372


---