X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=lib%2Fmemcached_storage.c;h=aac9b9227cae8d1c77f62ee0a912a86f53e39fb3;hb=dbce10442220e3c776226a3a2f4050cb6bcfbe42;hp=443389e029a583f03418a73f35695a39bef77f4c;hpb=8a86b578acc594d37a8638e3e0afba1286c4b6ca;p=m6w6%2Flibmemcached diff --git a/lib/memcached_storage.c b/lib/memcached_storage.c index 443389e0..aac9b922 100644 --- a/lib/memcached_storage.c +++ b/lib/memcached_storage.c @@ -53,17 +53,15 @@ static inline memcached_return memcached_send(memcached_st *ptr, char to_write; size_t write_length; ssize_t sent_length; - memcached_return rc; + memcached_return rc[MEMCACHED_MAX_REPLICAS]; char buffer[MEMCACHED_DEFAULT_COMMAND_SIZE]; unsigned int server_key; + uint8_t replicas= 0; - WATCHPOINT_ASSERT(!(value == NULL && value_length > 0)); - WATCHPOINT_ASSERT(!(value && value_length == 0)); - - if (key_length == 0) + unlikely (key_length == 0) return MEMCACHED_NO_KEY_PROVIDED; - if (ptr->number_of_hosts == 0) + unlikely (ptr->number_of_hosts == 0) return MEMCACHED_NO_SERVERS; if ((ptr->flags & MEM_VERIFY_KEY) && (memcachd_key_test(&key, &key_length, 1) == MEMCACHED_BAD_KEY_PROVIDED)) @@ -84,46 +82,59 @@ static inline memcached_return memcached_send(memcached_st *ptr, (unsigned long long)expiration, value_length); if (write_length >= MEMCACHED_DEFAULT_COMMAND_SIZE) - { - rc= MEMCACHED_WRITE_FAILURE; - goto error; - } - - rc= memcached_do(&ptr->hosts[server_key], buffer, write_length, 0); - if (rc != MEMCACHED_SUCCESS) - goto error; - - if ((sent_length= memcached_io_write(&ptr->hosts[server_key], value, value_length, 0)) == -1) - { - rc= MEMCACHED_WRITE_FAILURE; - goto error; - } + return MEMCACHED_WRITE_FAILURE; if ((ptr->flags & MEM_BUFFER_REQUESTS) && verb == SET_OP) to_write= 0; else to_write= 1; - if ((sent_length= memcached_io_write(&ptr->hosts[server_key], "\r\n", 2, to_write)) == -1) + do { - rc= MEMCACHED_WRITE_FAILURE; - goto error; - } + char response_buffer[MEMCACHED_DEFAULT_COMMAND_SIZE]; + + rc[replicas]= memcached_do(&ptr->hosts[server_key], buffer, write_length, 0); + + if (rc[replicas] != MEMCACHED_SUCCESS) + goto error; - if (to_write == 0) - return MEMCACHED_BUFFERED; + if ((sent_length= memcached_io_write(&ptr->hosts[server_key], value, value_length, 0)) == -1) + { + rc[replicas]= MEMCACHED_WRITE_FAILURE; + goto error; + } - rc= memcached_response(&ptr->hosts[server_key], buffer, MEMCACHED_DEFAULT_COMMAND_SIZE, NULL); + if ((sent_length= memcached_io_write(&ptr->hosts[server_key], "\r\n", 2, to_write)) == -1) + { + rc[replicas]= MEMCACHED_WRITE_FAILURE; + goto error; + } - if (rc == MEMCACHED_STORED) - return MEMCACHED_SUCCESS; - else - return rc; + if (to_write == 0) + return MEMCACHED_BUFFERED; + else + rc[replicas]= memcached_response(&ptr->hosts[server_key], + response_buffer, MEMCACHED_DEFAULT_COMMAND_SIZE, NULL); + /* On error we just jump to the next potential server */ error: - memcached_io_reset(&ptr->hosts[server_key]); + if (replicas > 1 && ptr->distribution == MEMCACHED_DISTRIBUTION_CONSISTENT) + { + if (server_key == (ptr->number_of_hosts - 1)) + server_key= 0; + else + server_key++; + } + } while ((++replicas) < ptr->number_of_replicas); + + /* As long as one object gets stored, we count this as a success */ + while (replicas--) + { + if (rc[replicas] == MEMCACHED_STORED) + return MEMCACHED_SUCCESS; + } - return rc; + return rc[0]; } memcached_return memcached_set(memcached_st *ptr, char *key, size_t key_length, @@ -206,7 +217,7 @@ memcached_return memcached_cas(memcached_st *ptr, memcached_return rc; rc= memcached_send(ptr, key, key_length, key, key_length, value, value_length, - expiration, flags, cas, APPEND_OP); + expiration, flags, cas, CAS_OP); return rc; } @@ -297,6 +308,6 @@ memcached_return memcached_cas_by_key(memcached_st *ptr, memcached_return rc; rc= memcached_send(ptr, key, key_length, key, key_length, value, value_length, - expiration, flags, cas, APPEND_OP); + expiration, flags, cas, CAS_OP); return rc; }