Permalink
Browse files

bug 1648 - need to set vbucket ID during single-key a2b commands

This should fix any single-key simple (non-item) ascii commands
(delete, incr/decr), including single-key ascii GET's.

The code was incorrectly not putting the vbucket ID into the request
opaque field.  During a not-my-vbucket response, the opaque field
was always providing a vbucket ID that was incorrectly 0.

Change-Id: I9ee65760685835cb83e941d03da94fa80f6353e8
Reviewed-on: http://review.northscale.com/972
Tested-by: Steve Yen <steve.yen@gmail.com>
Reviewed-by: Sean Lynch <seanl@literati.org>
  • Loading branch information...
1 parent 1af283b commit 8b453275bc54d2abeb0f23ddd83a85c6e0d7e9b5 @steveyen steveyen committed with seanlynch Jun 30, 2010
Showing with 1 addition and 0 deletions.
  1. +1 −0 cproxy_protocol_a2b.c
View
@@ -1308,6 +1308,7 @@ bool cproxy_forward_a2b_simple_downstream(downstream *d,
if (vbucket >= 0) {
header->request.reserved = htons(vbucket);
+ header->request.opaque = htonl(vbucket);
}
header->request.bodylen =

0 comments on commit 8b45327

Please sign in to comment.