Skip to content

Comments

delete returns NOT_FOUND as of 1.6.40#162

Closed
remicollet wants to merge 1 commit intoawesomized:v1.xfrom
remicollet:issue-161
Closed

delete returns NOT_FOUND as of 1.6.40#162
remicollet wants to merge 1 commit intoawesomized:v1.xfrom
remicollet:issue-161

Conversation

@remicollet
Copy link
Contributor

See #161

With memcached 1.6.40

$ src/bin/memcapable  -a -h localhost -p  11211
ascii version                           [pass]
ascii quit                              [pass]
ascii verbosity                         [pass]
ascii set                               [pass]
ascii set noreply                       [pass]
ascii get                               [pass]
ascii gets                              [pass]
ascii mget                              [pass]
ascii flush                             [pass]
ascii flush noreply                     [pass]
ascii add                               [pass]
ascii add noreply                       [pass]
ascii replace                           [pass]
ascii replace noreply                   [pass]
ascii cas                               [pass]
ascii cas noreply                       [pass]
ascii delete                            [pass]
ascii delete noreply                    [pass]
ascii incr                              [pass]
ascii incr noreply                      [pass]
ascii decr                              [pass]
ascii decr noreply                      [pass]
ascii append                            [pass]
ascii append noreply                    [pass]
ascii prepend                           [pass]
ascii prepend noreply                   [pass]
ascii stat                              [pass]
All tests passed

I add "NOT_FOUND" as a possible value for error

Other way is to expect a "NOT_FOUND" response, but this will make the test incompatible with older versions

@m6w6
Copy link
Collaborator

m6w6 commented Jan 26, 2026

Awesome! Thank you, Remi. Was wondering why it was failing on my sr.ht BSD builds lately.

@m6w6
Copy link
Collaborator

m6w6 commented Jan 26, 2026

Had to rebase it, since your commit was not signed; merged as 664fada

@m6w6 m6w6 closed this Jan 26, 2026
@remicollet remicollet deleted the issue-161 branch January 26, 2026 11:21
@remicollet
Copy link
Contributor Author

Had to rebase it, since your commit was not signed; merged as 664fada

Thanks

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.

2 participants