Jdesk / memcached

Automatically exported from code.google.com/p/memcached
0 stars 0 forks source link

1.3.2: ASCII set operations fail when run using UDP #36

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. apply the attached patch (udp.t.diff) to t/udp.t  

2. run the udp test (prove ./t/udp.t)

What is the expected output? What do you see instead?

-Expected: All tests should pass

-Actual: the set test fails and hangs

What version of the product are you using? On what operating system?

-1.3.2. running on Mac OS X (10.5.6) 

Please provide any additional information below.

-The problem appears to be due to the fact that the complete_nread() function 
does not handle 
the udp protocol (c->protocol == ascii_udp_protocol). Already have a patch, 
will submit it to 
the mailing list. 

-FWIW, glancing at the code suggest that there is no support for any UDP 
operations using 
binary protocol. Will create a test and file a bug as appropriate.

Original issue reported on code.google.com by eric.d.l...@gmail.com on 20 Mar 2009 at 10:58

Attachments:

GoogleCodeExporter commented 9 years ago
I fixed this based on your test and posting.

http://github.com/dustin/memcached/commit/9ce2e0ac5638064cb1d268292b08256e33643a
19

Original comment by dsalli...@gmail.com on 21 Mar 2009 at 9:22