Skip to content
Commits on Sep 24, 2010
  1. @steveyen

    initial printf only when -v

    steveyen committed
    Change-Id: Id67199c23a6acfe85d56c41044fe432ade99ba94
    Reviewed-on: http://review.northscale.com/2674
    Tested-by: Matt Ingenthron <matt@northscale.com>
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
  2. @steveyen

    more error msgs when invalid JSON

    steveyen committed
    Change-Id: Ie36f1bae0c2dc41ea694b2118adc6cc7cd39b97a
    Reviewed-on: http://review.northscale.com/2675
    Tested-by: Matt Ingenthron <matt@northscale.com>
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
  3. @steveyen

    usage() help tweaks

    steveyen committed with Matt Ingenthron
    Primarily added example of multiple URL's.
    
    Change-Id: I364b0580c23b60056ab2c0275abd60f0bdea4ce5
    Reviewed-on: http://review.northscale.com/2673
    Tested-by: Matt Ingenthron <matt@northscale.com>
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
Commits on Sep 23, 2010
  1. @steveyen

    hooked up more tests to 'make test'

    steveyen committed
    moxi_mock.pl script is now reusable, at the cost of some (perl
    inspired) readability.  It's now invoked to also run the mock auth,
    multitenancy, and multitenancy_default tests scripts (combinations of
    spawning a fake rest/http server, a moxi with the right params, and
    the right python test scripts to drive it).
    
    Change-Id: If7c16c9709ab59320d804909f578c7f9966a7a29
    Reviewed-on: http://review.northscale.com/2644
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
Commits on Sep 21, 2010
  1. @seanlynch @steveyen

    Bug 2266: Compare *entire* username and password.

    seanlynch committed with steveyen
    Change-Id: I35e254bf6c719124e3ade616ee901ea4e67059d6
    Reviewed-on: http://review.northscale.com/2221
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @mtaneja @steveyen

    Dynamic switching of downstream protocol

    mtaneja committed with steveyen
    Change-Id: Idf4fa06c071842d640034f303cbf44605586cbf5
    Reviewed-on: http://review.northscale.com/2595
    Tested-by: Steve Yen <steve.yen@gmail.com>
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
  3. @steveyen @seanlynch

    bug 2287 - handle binary ETMPFAIL response

    steveyen committed with seanlynch
    Change-Id: I06fbbd5e9cbc56972dc30b1a57c4d6dded37efe4
    Reviewed-on: http://review.northscale.com/2587
    Reviewed-by: Sean Lynch <seanl@literati.org>
    Tested-by: Sean Lynch <seanl@literati.org>
Commits on Sep 16, 2010
  1. @steveyen

    bug 2236 - don't printf unused dbpath anymore

    steveyen committed
    the local sqlite/dbpath feature of libconflate was confusing,
    so don't allude to it anymore in the -vvv output.
    
    Change-Id: I849f72128b4859cf69f0b1a1a3cde6a93a99211c
    Reviewed-on: http://review.northscale.com/2304
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @steveyen

    usage typo and width fix

    steveyen committed
    Change-Id: Ia77b17f9c9614434234bd4f4f5162416c46bddce
    Reviewed-on: http://review.northscale.com/2492
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
Commits on Sep 15, 2010
  1. @mtaneja @steveyen

    mcmux-prefixed "A:HOST:PORT version" command

    mtaneja committed with steveyen
    With tweaks so that it's not a broadcast command.
    
    Change-Id: Id3941efa4255d43cd012000c6be149bcf2a07f19
    Reviewed-on: http://review.northscale.com/2437
    Reviewed-by: Manik Taneja <mtaneja@zynga.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @steveyen

    bug 2345 - don't crash when some binary clients send in empty keys

    steveyen committed
    One client that does this is dbmaint, so you can test this fix by
    pointing dbmaint against moxi, and moxi doesn't crash anymore...
    
      ./ep-engine/management/dbmaint --vacuum --port=11211
    
    Change-Id: Ie1908333cb16d3b7a2f33c85ce5e34694db39de9
    Reviewed-on: http://review.northscale.com/2427
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
    Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  3. @trondn @steveyen

    Use the default automake rule to detect endian

    trondn committed with steveyen
    Change-Id: Ic09e0f2d3db93fdfd6d7d861476588ebacfc9a13
    Reviewed-on: http://review.northscale.com/2367
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  4. @trondn @steveyen

    Skip check for malloc and realloc

    trondn committed with steveyen
    Change-Id: I21ed07ed90e1ccc6fffd6b1336fb0ab6d37df7ba
    Reviewed-on: http://review.northscale.com/2366
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  5. @steveyen

    mcmux compat needs cmd-line behavior

    steveyen committed
    Change-Id: I2ced858d2e00e75a89f189c2b5cb9a4e8fa2059f
    Reviewed-on: http://review.northscale.com/2426
    Reviewed-by: Manik Taneja <mtaneja@zynga.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
Commits on Sep 14, 2010
  1. @steveyen

    bug 2337, moved MOXI_BLOCKING_CONNECT definition

    steveyen committed
    Moved to a place where windows can find it, as windows builder doesn't
    appear to use config_static.h.
    
    Change-Id: Ibca5f1b07b025ed58945a1959648324febbe67b5
    Reviewed-on: http://review.northscale.com/2419
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @mtaneja @steveyen

    misc changes for mcmux compatiblity

    mtaneja committed with steveyen
    1. version, quit and stats proxy are be treated as self commands
       and do not require a prefix
    2. gets should return SERVER_ERROR for a non existent server when
       running in mcmux compatiblity mode
    
    Change-Id: I3b9c231001955d80f4a8274c2f884c31cec2fe2e
    Reviewed-on: http://review.northscale.com/2417
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  3. @steveyen

    bug 2337, default to blocking connect() on windows platform

    steveyen committed
    Change-Id: Ic370eef54da59732aa38463c3de9cce48dab169b
    Reviewed-on: http://review.northscale.com/2412
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  4. @steveyen

    stats for connect error cases

    steveyen committed
    Change-Id: I3d499d69425a6693511e4e0eec8debe5c72f4292
    Reviewed-on: http://review.northscale.com/2403
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  5. @steveyen

    check for downstream_conn_max to limit number of connections

    steveyen committed
    If moxi reaches a max number of downstream connections (per host_ident
    per thread), the zstored_acquire_downstream_conn() function will
    return that condition (via an extra out param in its API).
    
    After that, the downstream has to wait until there's an available
    downstream_conn, which it does using the new
    zstored_downstream_waiting_add() function (and all the related,
    supporting downstream_waiting_head/tail linked-list/queue management
    code).
    
    As a downstream conn is released, moxi assigns the downstream conn to
    a waiting downstream.
    
    When all downstream conns are closed (for a given host_ident &
    thread), moxi processes any waiting downstreams.
    
    Change-Id: I52e754069a959f319aca545f7de33470892681cc
    Reviewed-on: http://review.northscale.com/2402
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  6. @steveyen

    downstream_waiting_head/tail started

    steveyen committed
    Change-Id: I061361b34774ec4493383eb9e00a1b988b65e0a2
    Reviewed-on: http://review.northscale.com/2401
    Reviewed-by: Jayesh Jose <jjose@zynga.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  7. @steveyen

    added downstream_conn_max config parsing

    steveyen committed
    Change-Id: Icef738e732eae3b1205bcdfb50d59b8da6e4c2f6
    Reviewed-on: http://review.northscale.com/2400
    Reviewed-by: Jayesh Jose <jjose@zynga.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
Commits on Sep 13, 2010
  1. @steveyen

    windows needs EINPROGRESS, EISCONN

    steveyen committed
    Change-Id: I5295bd136a9b8cf65d5ef0feaaa54dfccec81cb4
    Reviewed-on: http://review.northscale.com/2312
    Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @steveyen

    bug 2237 - calling zstored_error_count() more during errors

    steveyen committed
    A zstored_error_count() invocation was incorrectly being guarded by an
    IF-THEN statement.  So, this fix moves the call out of the IF body,
    but doing so meant having to change zstored_error_count()'s function
    signature (to just take a simple host_ident string).
    
    Also, refactored out a zstored_get_downstream_conns() helper function,
    to handle a case when the zstored_error_count() wasn't counting
    correctly the very first time before moxi had tried its very first
    connection attempt.
    
    Change-Id: I08468db8d767ea555c893d6c39276dc93968b92e
    Reviewed-on: http://review.northscale.com/2378
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  3. @steveyen

    connect_max_errors/retry config in stats proxy

    steveyen committed with Matt Ingenthron
    Change-Id: Id1b5df1fb44079fafa1ae67b5beea56840be71cc
    Reviewed-on: http://review.northscale.com/2377
    Reviewed-by: Matt Ingenthron <matt@northscale.com>
    Tested-by: Matt Ingenthron <matt@northscale.com>
Commits on Sep 10, 2010
  1. @trondn @dustin

    Add .libs to list of ignored files

    trondn committed with dustin
    Change-Id: I7137473043bcad450d6347b5f6d114beb611799f
    Reviewed-on: http://review.northscale.com/2365
    Reviewed-by: Dustin Sallings <dustin@spy.net>
    Tested-by: Dustin Sallings <dustin@spy.net>
Commits on Sep 9, 2010
  1. @steveyen

    conn might be NULL during debug logging

    steveyen committed
    ...especially for a broadcast command (like stats or flush_all).
    
    Change-Id: I493af84c96bc1e5098b52dc336281aa22a2e32c9
    Reviewed-on: http://review.northscale.com/2302
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @steveyen

    from zstored, 5 sec timeout during non-blocking connect()

    steveyen committed
    As part of this change, refactored out a update_event_timed() helper
    function.
    
    Change-Id: Idae1834c21cfb35e1030bc1e3f33b9d65a1be46a
    Reviewed-on: http://review.northscale.com/2301
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
Commits on Sep 8, 2010
  1. @mtaneja @steveyen

    fix support for unix domain sockets

    mtaneja committed with steveyen
    Change-Id: Id9efc18e2e5a658bf198cd37b5852a11f1f2a1ef
    Reviewed-on: http://review.northscale.com/2320
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  2. @mtaneja @steveyen

    mcmux protocol support for moxi

    mtaneja committed with steveyen
    Added support for ASCII protocol extension to moxi. To enable
    mcmux compatiblity -X option should be passed as one of the command
    line paramters along with the port number that moxi should listen on.
    
    ./moxi -p 11311 -X
    
    when started in this mode, moxi expects the mcmux protocol string
    to be appended before the start of a memcache command.
    An example of an interaction between client and moxi :
    
    A:localhost:11411 get k1
    VALUE k1 0 5
    manik
    END
    
    get k1
    ERROR
    
    Amended version: check for null d->upstream_conn
    Use a stack variable for msst instead of allocating it.
    
    Change-Id: I40d853023c80d08a40113e338e5bb6708e7f4bd9
    Reviewed-on: http://review.northscale.com/2317
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  3. @steveyen

    connect timeout and retry interval are now runtime configurable

    steveyen committed
    Usage...
    
      ./moxi -Z connect_max_errors=10,connect_retry_interval=10000,other_flags=here
    
    Change-Id: I8de9e94e9f6e23fa9a8214add476af699c423616
    Reviewed-on: http://review.northscale.com/2289
    Tested-by: Steve Yen <steve.yen@gmail.com>
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
  4. @steveyen

    checking MAX_DOWNSTREAM_CONNECTION_ERRORS and DOWNSTREAM_RETRY_INTERVAL

    steveyen committed
    Change-Id: I2c3574f5564a45cdf916c66261759d6d2276b605
    Reviewed-on: http://review.northscale.com/2288
    Tested-by: Steve Yen <steve.yen@gmail.com>
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
  5. @steveyen

    fixed connection leak

    steveyen committed
    Change-Id: I6542dc05d1569365961e872fe9420dc7eeca6244
    Reviewed-on: http://review.northscale.com/2283
    Reviewed-by: Manik Taneja <mtaneja@zynga.com>
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  6. @steveyen

    tracking connect_failed stat

    steveyen committed
    Change-Id: Ib783cae1283845909a473a2df08e4a6643037f9a
    Reviewed-on: http://review.northscale.com/2280
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  7. @steveyen

    more assert()

    steveyen committed
    Change-Id: Ib073e89cd54ca3b5fca8a5fd0631969d3953154b
    Reviewed-on: http://review.northscale.com/2279
    Reviewed-by: Dustin Sallings <dustin@spy.net>
    Tested-by: Steve Yen <steve.yen@gmail.com>
  8. @steveyen

    removed unused downstream_waiting/next_waiting

    steveyen committed
    Change-Id: I453b6865463035934e364dc74eeb69919ef1a5d2
    Reviewed-on: http://review.northscale.com/2278
    Tested-by: Steve Yen <steve.yen@gmail.com>
    Reviewed-by: Steve Yen <steve.yen@gmail.com>
Something went wrong with that request. Please try again.