GitHub user iostat opened a pull request:

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

    [STORM-1375] Blobstore broke Pacemaker

    When using the new Pacemaker (which, you kind of have to, cos the 
alternative of not using it is abusing ZK to the point of deadlocking it...), 
you cannot submit topologies because submitTopology is calling ClusterState's 
delete_node_blobstore which never got implemented in 
pacemaker_state_factory.clj.
    
    This PR introduces a noop implementation of default-node-blobstore for 
pacemaker.

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

    $ git pull https://github.com/socialrank/storm pacemaker-blobstore

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

    https://github.com/apache/storm/pull/926.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 #926
    
----
commit d14dabdafc9c7c9f5f1b97eae522a0373fe38669
Author: Ilya Ostrovskiy <[email protected]>
Date:   2015-12-05T15:29:36Z

    default noop implementation for delete-node-blobstore for 
pacemaker_state_factory

commit 2246b72224b418343958f95762ebfdc75c9ab044
Author: Ilya Ostrovskiy <[email protected]>
Date:   2015-12-05T15:32:58Z

    move delete-node-blobstore lower so its not implied it passes through to
    zk-state

----


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