GitHub user bastiliu opened a pull request:

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

    [STORM-1245] port backtype.storm.daemon.acker to java

    1. Increase bucket number of pending map from 2 to 3 to avoid timeout too 
fast.
    2. In clojure testing, creation of acker bolt will be overridden in 
with-tracked-cluster. After porting the implementation of acker to java, it 
seems there is no way to do similar override for java method. So, the 
mk-acker-bolt of acker.clj was kept as temporary solution to make the clojure 
testing pass. It can be removed after the porting of common.clj and relative 
testing modules. 
    Anyway, if there is a solution or workaround for this issue, please help to 
comment.

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

    $ git pull https://github.com/bastiliu/storm master

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

    https://github.com/apache/storm/pull/1069.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 #1069
    
----
commit e9dc271f11e311eea2269a8f6035e2c322d8d520
Author: basti.lj <[email protected]>
Date:   2016-02-01T03:17:05Z

    [STORM-1245] port backtype.storm.daemon.acker to java

commit 675b0c4f786838a13122b6743ca6c946aa1d63ee
Author: basti.lj <[email protected]>
Date:   2016-02-03T07:46:08Z

    [Storm 1245] port backtype.storm.daemon.acker to java

commit e50a312f1440131e8b9e0cc055d475cbbe711cb9
Author: basti.lj <[email protected]>
Date:   2016-02-03T08:01:43Z

    [STORM-1245] port backtype.storm.daemon.acker to java

----


---
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.
---

Reply via email to