Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

store_message should overwrite any existing records #535

Closed
bbangert opened this issue Jul 19, 2016 · 0 comments
Closed

store_message should overwrite any existing records #535

bbangert opened this issue Jul 19, 2016 · 0 comments
Assignees
@bbangert
Labels
enhancement
Milestone
PUSHSVC-0: quality

Comments

@bbangert
Copy link
Member

Sentry Error:

ConditionalCheckFailedException: ConditionalCheckFailedException: 400 Bad Request
{u'message': u'The conditional request failed', u'__type': u'com.amazonaws.dynamodb.v20120810#ConditionalCheckFailedException'}
@bbangert bbangert added this to the PUSHSVC-0: quality milestone Jul 19, 2016
@bbangert bbangert self-assigned this Jul 19, 2016
bbangert added a commit that referenced this issue Jul 19, 2016
@bbangert
fix: overwrite existing messages instead of throwing an error
f5d3c9a 
DynamoDB throws a condition check failure if you put_item and
it already exists. This can occur when we save a message that
may already exist when a user hops servers without ack'ing. We
should allow it since it consumes the same resources.

Closes #535
@bbangert bbangert mentioned this issue Jul 19, 2016
Merged
bbangert added a commit that referenced this issue Jul 19, 2016
@bbangert
fix: overwrite existing messages instead of throwing an error
aa73881 
DynamoDB throws a condition check failure if you put_item and
it already exists. This can occur when we save a message that
may already exist when a user hops servers without ack'ing. We
should allow it since it consumes the same resources.

Closes #535
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bbangert bbangert

Labels
enhancement
Projects
None yet
Milestone
PUSHSVC-0: quality
Development

No branches or pull requests
2 participants
@jrconlin @bbangert