Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove a couple of TODOs, make the read buffer size configurable, and don't ↵bug26457 | Simon MacMullen | 2014-11-13 | 1 | -1/+2 |
| | | | | use the read buffer for the QI or msg store transform since they already read in decent sized chunks. | ||||
* | Switch to Erlang 17-style types, /0 versions at least. | Simon MacMullen | 2014-09-17 | 1 | -2/+2 |
| | |||||
* | Update copyright for 2014bug25940 | Simon MacMullen | 2014-03-17 | 1 | -1/+1 |
| | |||||
* | correctness++bug26066 | Matthias Radestock | 2014-03-13 | 1 | -1/+1 |
| | |||||
* | turns out that cleaning up 'clients' in client_terminate is bad | Matthias Radestock | 2014-03-13 | 1 | -6/+6 |
| | | | | | since it breaks clean shutdown/recovery, which relies on the CRefs in 'clients'. | ||||
* | invoke credit_flow:peer_down on, err, DOWN. *only* | Matthias Radestock | 2014-03-13 | 1 | -8/+11 |
| | | | | | | | | Thus preventing potential leaks and deadlocks. Also, clean up 'clients' in clear_client, and thus client_terminate, which was previously missing and thus the source of another potential leak. | ||||
* | s/VMware/GoPivotal/g | Simon MacMullen | 2013-07-01 | 1 | -2/+2 |
| | |||||
* | cosmetic: state the not-immediatly-obvious | Matthias Radestock | 2013-06-17 | 1 | -1/+8 |
| | |||||
* | Throw the key too. And if the server side still fails try to fail with more ↵bug25611 | Simon MacMullen | 2013-06-17 | 1 | -2/+3 |
| | | | | information. | ||||
* | Check value when clients update flying_ets | Emile Joubert | 2013-06-17 | 1 | -1/+4 |
| | |||||
* | Merged stable into default | Emile Joubert | 2013-04-10 | 1 | -13/+10 |
|\ | |||||
| * | Merge in default | Simon MacMullen | 2013-01-30 | 1 | -9/+6 |
| |\ | |||||
| | * | stable to default | Simon MacMullen | 2013-01-24 | 1 | -1/+1 |
| | |\ | |||||
| | * | | rationalise timer maintenance | Matthias Radestock | 2013-01-08 | 1 | -8/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - introduce a couple of helper functions and use them wherever we can - pay attention to the result of timer cancellation, to prevent duplicate timer creation - stop all timers when a queue terminates. More out of politeness than necessity. | ||||
| * | | | Pass length to prioritise_* | Simon MacMullen | 2012-11-08 | 1 | -5/+5 |
| |/ / | |||||
* | | | Only queue up for GC at most two pairs of files at once, since we lock files ↵bug25514 | Simon MacMullen | 2013-04-05 | 1 | -3/+8 |
| |/ |/| | | | | | as soon as they are queued; no point in having them locked for ages as they wait to get GCed. | ||||
* | | Update copyright 2013bug25343 | Emile Joubert | 2013-01-23 | 1 | -1/+1 |
|/ | |||||
* | file_name -> filename | Matthias Radestock | 2012-09-17 | 1 | -5/+5 |
| | | | | | | | | | The latter is used in more places and also by OTP. In the process fix a bug - rabbit_upgrade was referencing rabbit_node_monitor:running_nodes_filename/0, which was in fact named running_nodes_file_name. That function was also missing a spec. And it was in the "wrong" place w.r.t. the function order in the module ;) | ||||
* | cosmetic | Matthias Radestock | 2012-04-11 | 1 | -2/+2 |
| | |||||
* | Changed uses of gb_sets:difference/2 to our difference functions. | Francesco Mazzoli | 2012-04-11 | 1 | -1/+2 |
| | |||||
* | Update copyright 2012bug24606 | Emile Joubert | 2012-02-02 | 1 | -1/+1 |
| | |||||
* | More credit for the queue -> msg_store case. | Simon MacMullen | 2012-01-19 | 1 | -2/+2 |
| | |||||
* | two flavours of 'write' | Matthias Radestock | 2012-01-17 | 1 | -11/+19 |
| | | | | | | | Just like in the other applications of credit_flow:send/ack, we need two flavours of the surrounding API function, i.e. 'write' and 'write_flow' in this case. That way existing call sites, such as in rabbit_test can remain undisturbed. | ||||
* | more sensible arg order | Matthias Radestock | 2012-01-17 | 1 | -13/+13 |
| | | | | and consistent var naming | ||||
* | Don't pass the client pid in for write. | Simon MacMullen | 2012-01-17 | 1 | -3/+5 |
| | |||||
* | Flow control: vq -> msg_store. | Simon MacMullen | 2012-01-17 | 1 | -17/+26 |
| | |||||
* | more accurate description of flying_ets tablebug24308 | Matthias Radestock | 2011-10-17 | 1 | -1/+1 |
| | |||||
* | explain why client_update_flying phase 3 will always succeed | Matthias Radestock | 2011-10-17 | 1 | -0/+3 |
| | |||||
* | refactor: revert change to var name | Matthias Radestock | 2011-10-17 | 1 | -2/+2 |
| | |||||
* | refactor: rename update_flying/3 to client_update_flying/3 | Matthias Radestock | 2011-10-17 | 1 | -4/+4 |
| | |||||
* | fix bug | Matthias Radestock | 2011-10-17 | 1 | -1/+1 |
| | |||||
* | Give up on attempting to improve clarity. | Simon MacMullen | 2011-10-17 | 1 | -6/+6 |
| | |||||
* | Rephrase to make Matthias happier... | Simon MacMullen | 2011-10-17 | 1 | -7/+7 |
| | |||||
* | I find this clearer | Simon MacMullen | 2011-10-17 | 1 | -6/+6 |
| | |||||
* | rename 'removed' confirmation event name to 'ignored' | Matthias Radestock | 2011-10-15 | 1 | -3/+3 |
| | | | | since that reflects its meaning better | ||||
* | merge default into bug24308 | Matthias Radestock | 2011-10-15 | 1 | -13/+8 |
|\ | |||||
| * | a spot of inlining | Matthias Radestock | 2011-10-15 | 1 | -14/+7 |
| | | |||||
| * | only confirm messages on 'remove' that are pending confirmation | Matthias Radestock | 2011-10-11 | 1 | -2/+3 |
| | | |||||
* | | documentation | Matthias Radestock | 2011-10-14 | 1 | -0/+39 |
| | | |||||
* | | issue confirms when needed (and only then) | Matthias Radestock | 2011-10-11 | 1 | -22/+33 |
| | | | | | | | | | | | | This also fixes a bug that is present on 'default', namely that we used to always issue a confirm on 'remove' regardless of whether there was preceding 'write'. | ||||
* | | refactoring: extract core of 'write' handler | Matthias Radestock | 2011-10-10 | 1 | -66/+62 |
| | | | | | | | | | | | | similar to what we already did for 'remove' Also, don't add the 'dying client' marker message to the pending confirms. While that does no harm, it is clearly not right. | ||||
* | | cosmetic | Matthias Radestock | 2011-10-10 | 1 | -2/+2 |
| | | |||||
* | | there's more than one call site for remove_message... | Matthias Radestock | 2011-10-10 | 1 | -41/+41 |
| | | | | | | | | ...and only one of them must pay attention to flying_ets | ||||
* | | be more careful about deleting cur_file_cache_ets entries | Matthias Radestock | 2011-10-10 | 1 | -6/+17 |
| | | |||||
* | | I'm pretty sure this is correct | Matthias Radestock | 2011-10-09 | 1 | -31/+47 |
| | | |||||
* | | slightly more obvious flying_writes accounting | Matthias Radestock | 2011-10-06 | 1 | -14/+11 |
| | | | | | | | | | | and move the cache cleanup from 'remove' to 'write', which is a more obvious place. Improve docs on that too. | ||||
* | | merge default into bug24308 | Matthias Radestock | 2011-10-06 | 1 | -73/+116 |
|\ \ | |/ |/| | |||||
| * | Whoops, forgot that bit. | Matthew Sackman | 2011-08-19 | 1 | -0/+4 |
| | | |||||
| * | Fixed again. | Matthew Sackman | 2011-08-19 | 1 | -73/+112 |
| | | |||||
* | | move file functions from misc to rabbit_file | Alexandru Scvortov | 2011-09-21 | 1 | -4/+4 |
| | |