Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge bug25205 | Simon MacMullen | 2012-10-19 | 2 | -1/+14 |
|\ | |||||
| * | complain when running 'proper' and there are no tests | Matthias Radestock | 2012-10-19 | 1 | -0/+9 |
| | | | | | | | | Twice already I wasted way too much time tracking this down. | ||||
| * | Log Rabbit / Erlang versions on startup, and fix the stop message.bug25230 | Simon MacMullen | 2012-10-18 | 1 | -1/+4 |
| | | |||||
* | | reinstate rabbit_vm:memory/0, accidentally stripped out whilst reverting changesbug25205 | Tim Watson | 2012-10-18 | 1 | -1/+1 |
| | | |||||
* | | wrap the whole start after all, and use local_info_msg/2 to avoid duplication | Tim Watson | 2012-10-18 | 1 | -28/+27 |
| | | |||||
* | | merge default | Tim Watson | 2012-10-17 | 2 | -10/+30 |
|\ \ | |/ |/| | |||||
| * | revert to a79f54b909f | Tim Watson | 2012-10-17 | 1 | -25/+26 |
| | | |||||
| * | include the error term in start/boot exits and refactor the tests to handle ↵ | Tim Watson | 2012-10-08 | 2 | -13/+25 |
| | | | | | | | | 'EXIT' instead of 'throw' for start/boot failures | ||||
| * | wrap the whole of start/boot in the same error check | Tim Watson | 2012-10-08 | 1 | -17/+11 |
| | | |||||
| * | don't try too hard to get the output errors looking nice | Tim Watson | 2012-10-05 | 1 | -7/+0 |
| | | |||||
| * | start log handlers first, so we see the output only once | Tim Watson | 2012-10-05 | 1 | -2/+2 |
| | | |||||
| * | also handle cluster status file prep failures | Tim Watson | 2012-10-05 | 1 | -6/+10 |
| | | |||||
| * | oops | Tim Watson | 2012-10-05 | 1 | -8/+9 |
| | | |||||
| * | treat cluster consistency check failures as boot errors | Tim Watson | 2012-10-05 | 1 | -3/+18 |
| | | |||||
* | | Fix formatted error handling as broken by Matthias in 891e659f0e77. | Simon MacMullen | 2012-10-16 | 2 | -6/+8 |
| | | |||||
* | | cosmetic | Matthias Radestock | 2012-10-16 | 1 | -4/+2 |
| | | |||||
* | | one less place to equate mirrors with nodes | Matthias Radestock | 2012-10-16 | 3 | -9/+8 |
| | | |||||
* | | cosmeticbug25202 | Matthias Radestock | 2012-10-16 | 3 | -7/+4 |
| | | |||||
* | | merge default into bug25202 | Matthias Radestock | 2012-10-16 | 2 | -127/+78 |
|\ \ | |||||
| * \ | merge heads | Matthias Radestock | 2012-10-16 | 1 | -5/+7 |
| |\ \ | |||||
| | * | | compare pids instead of nodes in slave's handle_call/gm_deaths | Matthias Radestock | 2012-10-16 | 1 | -5/+7 |
| | | | | |||||
| * | | | Merge bug25225 | Simon MacMullen | 2012-10-16 | 2 | -102/+55 |
| |\ \ \ | | |/ / | |/| | | |||||
| | * | | merge slave 'confirm' decision making and actionbug25225 | Matthias Radestock | 2012-10-15 | 1 | -31/+21 |
| | | | | | | | | | | | | | | | | similar to what we did in rabbit_amqqueue_process | ||||
| | * | | refactor: handle all msg statuses uniformly | Matthias Radestock | 2012-10-15 | 1 | -6/+1 |
| | | | | |||||
| | * | | refactor: deal with 'publish' and 'publish_delivered' in separate clauses | Matthias Radestock | 2012-10-15 | 1 | -12/+11 |
| | | | | |||||
| | * | | refactor: extract commonality between 'publish' and 'discard' | Matthias Radestock | 2012-10-15 | 1 | -73/+30 |
| | | | | |||||
| | * | | deal with confirms in slave 'discard' | Matthias Radestock | 2012-10-15 | 1 | -2/+14 |
| | | | | |||||
| | * | | merge default into bug25225 | Matthias Radestock | 2012-10-15 | 1 | -11/+9 |
| | |\ \ | |||||
| | * \ \ | merge default into bug25225 | Matthias Radestock | 2012-10-12 | 3 | -79/+61 |
| | |\ \ \ | |||||
| | * | | | | optimise x-message-ttl=0 & confirms path on queue | Matthias Radestock | 2012-10-12 | 1 | -14/+10 |
| | | | | | | | | | | | | | | | | | | | | | | | | changes to slave still to come | ||||
| * | | | | | remove comment that doesn't make any sense and is irrelevant | Matthias Radestock | 2012-10-16 | 1 | -2/+0 |
| | | | | | | |||||
| * | | | | | merge heads | Matthias Radestock | 2012-10-16 | 1 | -1/+1 |
| |\ \ \ \ \ | |||||
| | * | | | | | Typo | Simon MacMullen | 2012-10-16 | 1 | -1/+1 |
| | | | | | | | |||||
| * | | | | | | refactor: don't track slave's master_pid separately | Matthias Radestock | 2012-10-16 | 1 | -18/+16 |
| |/ / / / / | |||||
* | | | | | | Make sure we don't stomp on GMPids if a slave manages to get to Mnesia ↵ | Simon MacMullen | 2012-10-16 | 1 | -3/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | before us. | ||||
* | | | | | | We don't need the 'existing' records, and without them we don't need to ↵ | Simon MacMullen | 2012-10-16 | 2 | -7/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | compare GMPids for sameness. | ||||
* | | | | | | simplify | Matthias Radestock | 2012-10-16 | 1 | -4/+1 |
| | | | | | | |||||
* | | | | | | merge default into bug25202 | Matthias Radestock | 2012-10-16 | 12 | -162/+309 |
|\ \ \ \ \ \ | |/ / / / / | | | | | | | | | | | | | | | | | | | ...and in the process inline mq_slave:forget_slave and fix a bug in it (it was looking at the wrong element of the tuple) | ||||
| * | | | | | refactor: slightly more sensible signature for add_slave | Matthias Radestock | 2012-10-16 | 1 | -6/+8 |
| | | | | | | |||||
| * | | | | | refactor: correct confusing variable naming | Matthias Radestock | 2012-10-16 | 1 | -6/+6 |
| | | | | | | |||||
| * | | | | | merge default (again)bug25207 | Tim Watson | 2012-10-15 | 1 | -11/+9 |
| |\ \ \ \ \ | | | |_|/ / | | |/| | | | |||||
| | * | | | | plug leakbug25226 | Matthias Radestock | 2012-10-13 | 1 | -11/+9 |
| | | |/ / | | |/| | | | | | | | | | | | | | | | | | always remove from PendingCh when we receive a msg via gm that we previously received from the channel. | ||||
| * | | | | merge default | Tim Watson | 2012-10-15 | 11 | -211/+344 |
| |\ \ \ \ | | |/ / / | |||||
| | * | | | correct essay & comments regarding the 'delayed confirm' rationale | Matthias Radestock | 2012-10-12 | 2 | -23/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Matthew has confirmed that the "we don't know the msg_seq_no until we receive the msg from the channel" reason is bogus. The msg_seq_no is allocated by the channel prior to routing and thus is the same across the master and all slaves. Hence the 'publish' via gm contains all the information we need to issue a confirm. Nevertheless we cannot actually issue the confirm until we've received the message from the channel. The essay now explains the real reason. | ||||
| | * | | | store less in the slave's msg_id_status | Matthias Radestock | 2012-10-12 | 1 | -20/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no need to track the ChPid of 'published' and 'confirmed'. Also: neater conversion of msg_id_status on promotion. | ||||
| | * | | | refactor: simplify confirm handling in queue | Matthias Radestock | 2012-10-12 | 1 | -31/+19 |
| | | | | | | | | | | | | | | | | | | | | three functions into one | ||||
| | * | | | cosmetic | Matthias Radestock | 2012-10-12 | 1 | -3/+2 |
| | | | | | |||||
| | * | | | re-instate TODO | Matthias Radestock | 2012-10-12 | 1 | -0/+1 |
| | | | | | |||||
| | * | | | a spot of inlining | Matthias Radestock | 2012-10-12 | 1 | -15/+10 |
| | |/ / | | | | | | | | | | | | | | | | | ...and refactoring; exploiting the fact that maybe_record_confirm_message is a no-op when Confirm == never | ||||
| | * | | Merge heads | Simon MacMullen | 2012-10-12 | 1 | -2/+4 |
| | |\ \ |