Prevent badarith when memory is unavailable. #111

Merged
merged 1 commit into from Jul 1, 2013

Conversation

Projects
None yet
3 participants
@cmeiklejohn
Contributor

cmeiklejohn commented Jul 1, 2013

When a node becomes reachable directly after starting and memsup is
still starting, memory can be unavailable. Prevent this computation
until it's available, avoding crashing webmachine with a badarith.

Prevent badarith when memory is unavailable.
When a node becomes reachable directly after starting and memsup is
still starting, memory can be unavailable.  Prevent this computation
until it's available, avoding crashing webmachine with a badarith.
@seancribbs

This comment has been minimized.

Show comment Hide comment
@seancribbs

seancribbs Jul 1, 2013

👍

👍

@cmeiklejohn

This comment has been minimized.

Show comment Hide comment
@cmeiklejohn

cmeiklejohn Jul 1, 2013

Contributor
@jaredmorrow

This comment has been minimized.

Show comment Hide comment
@jaredmorrow

jaredmorrow Jul 1, 2013

Contributor

Seems like something that should go in and is relatively low risk. If @seancribbs is +1, I am as well. I will bump the tag so it gets included in rc2

Contributor

jaredmorrow commented Jul 1, 2013

Seems like something that should go in and is relatively low risk. If @seancribbs is +1, I am as well. I will bump the tag so it gets included in rc2

@cmeiklejohn

This comment has been minimized.

Show comment Hide comment
@cmeiklejohn

cmeiklejohn Jul 1, 2013

Contributor

OK, I'm going to merge this.

Contributor

cmeiklejohn commented Jul 1, 2013

OK, I'm going to merge this.

cmeiklejohn added a commit that referenced this pull request Jul 1, 2013

Merge pull request #111 from basho/csm-prevent-bad-arith
Prevent badarith when memory is unavailable.

@cmeiklejohn cmeiklejohn merged commit 9799b99 into 1.4 Jul 1, 2013

1 check passed

default The Travis CI build passed
Details

@cmeiklejohn cmeiklejohn deleted the csm-prevent-bad-arith branch Jul 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment