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

Added support for update an existing value not present in cache. #5

Merged
merged 2 commits into from Sep 26, 2013
Merged

Added support for update an existing value not present in cache. #5

merged 2 commits into from Sep 26, 2013

Conversation

valeriangalliat
Copy link
Contributor

Pull request for issue #4, replacing the UPDATE or INSERT statements regarding of the presence of the key in cache by a single REPLACE statement.

Also updated tests so the case when a fresh Store sets an existing key without having it in cache.

@mthenw
Copy link
Owner

mthenw commented Sep 26, 2013

Will you resolve those conflicts?

@valeriangalliat
Copy link
Contributor Author

I updated my repo, everything seems ok (sorry I'm not used to this fork/pull/merge stuff)

mthenw added a commit that referenced this pull request Sep 26, 2013
Added support for update an existing value not present in cache.
@mthenw mthenw merged commit b0b2375 into mthenw:master Sep 26, 2013
@mthenw
Copy link
Owner

mthenw commented Sep 26, 2013

Thanks for this PR 👍 In few minutes there will be new version :)

@mthenw
Copy link
Owner

mthenw commented Sep 26, 2013

version 0.3.0 is in packagist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants