kv_wm_utils expects ?MD_DELETED to be "true" not 'true' #532

Merged
merged 1 commit into from Apr 19, 2013

Conversation

Projects
None yet
2 participants
@jrwest
Contributor

jrwest commented Apr 19, 2013

minimal possible fix. long term these types of things should be encapsulated
in riak_object

found here (includes steps to recreate): #520 (comment) (Note: to recreate issue you must create a tombstone sibling, not just a sibling)

Java code that exposes issue: https://gist.github.com/broach/57ce9237ec32b47d8897

cc/ @broach @seancribbs

kv_wm_utils expects ?MD_DELETED to be "true" not 'true'
minimal possible fix. long term these types of things should be encapsulated
in riak_object

broach pushed a commit that referenced this pull request Apr 19, 2013

Brian Roach
Reverted to checking for string instead of atom
The change in #532 makes this a string again ("true")

@ghost ghost assigned seancribbs Apr 19, 2013

@seancribbs

This comment has been minimized.

Show comment Hide comment
@seancribbs

seancribbs Apr 19, 2013

Contributor

👍 This fixes the problem.

Contributor

seancribbs commented Apr 19, 2013

👍 This fixes the problem.

jrwest added a commit that referenced this pull request Apr 19, 2013

Merge pull request #532 from basho/jrw-md-deleted-fix
kv_wm_utils expects ?MD_DELETED to be "true" not 'true'

@jrwest jrwest merged commit 561be08 into master Apr 19, 2013

1 check failed

default The Travis build failed
Details

@jrwest jrwest deleted the jrw-md-deleted-fix branch Apr 19, 2013

broach pushed a commit that referenced this pull request May 7, 2013

Reverted to checking for string instead of atom
The change in #532 makes this a string again ("true")

broach pushed a commit that referenced this pull request May 15, 2013

Reverted to checking for string instead of atom
The change in #532 makes this a string again ("true")

@seancribbs seancribbs removed their assignment May 8, 2015

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