Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ring events monitoring by riak_core_node_watcher #399

Closed
wants to merge 8 commits into from

Commits on Sep 27, 2013

  1. Fixes #343

    Before this patch, the riak_core_node_watcher.erl code assumed
    that if the riak_core_ring_events proc (a gen_event server) died,
    then a {gen_event_EXIT,_,_} message would be sent.  However, that
    assumption is not correct.  If it dies, we get a regular {EXIT,_,_}
    message.
    
    Tested by repeated use of alternating:
    * exit(whereis(riak_core_ring_events), kill).
    
    ... and looking at the length of the links list
    of process_info(whereis(riak_core_ring_events), links) -- it should
    be three, not two.
    slfritchie committed Sep 27, 2013
    Copy the full SHA
    6be124b View commit details
    Browse the repository at this point in the history

Commits on Oct 1, 2013

  1. Address review comments

    slfritchie committed Oct 1, 2013
    2
    Copy the full SHA
    33c322d View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    bbcc3a3 View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2013

  1. Copy the full SHA
    01c0ada View commit details
    Browse the repository at this point in the history

Commits on Oct 15, 2013

  1. Copy the full SHA
    c8986a3 View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2013

  1. Copy the full SHA
    65ee0a4 View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    4560248 View commit details
    Browse the repository at this point in the history
  3. Copy the full SHA
    6e4f156 View commit details
    Browse the repository at this point in the history