function memcached_behavior_set_distribution()

[ping,cai]

memcached_return_t memcached_behavior_set_distribution(memcached_st *ptr, memcached_server_distribution_t type)
{
  if (type < MEMCACHED_DISTRIBUTION_CONSISTENT_MAX)
  {
    if (MEMCACHED_DISTRIBUTION_CONSISTENT_WEIGHTED) // is here wrong??
    {
      ptr->ketama.weighted= true;
    }
    else
    {
      ptr->ketama.weighted= false;
    }

    ptr->distribution= type;
    return run_distribution(ptr);
  }

  return memcached_set_error(*ptr, MEMCACHED_INVALID_ARGUMENTS, MEMCACHED_AT,
                             memcached_literal_param("Invalid memcached_server_distribution_t"));
}

should if (MEMCACHED_DISTRIBUTION_CONSISTENT_WEIGHTED)
 be if (type == MEMCACHED_DISTRIBUTION_CONSISTENT_WEIGHTED)

[Brian Aker (brianaker)]

Hi,

On Jun 6, 2012, at 2:27 PM, ping,cai wrote:

> if (MEMCACHED_DISTRIBUTION_CONSISTENT_WEIGHTED) // is here wrong??

All weights are set to be the same by default, so while the above is a bug,… behavior is identical.

Cheers,
 -Brian

[Brian Aker (brianaker)]

Fixed in 1.0.9