Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Refactor the Appservice scheduler code (#5886) | Richard van der Hoff | 2019-08-20 | 1 | -45/+65 |
|\ | | | | | | | | | | | | | Get rid of the labyrinthine `recoverer_fn` code, and clean up the startup code (it seemed to be previously inexplicably split between `ApplicationServiceScheduler.start` and `_Recoverer.start`). Add some docstrings too. | ||||
| * | Refactor the Appservice scheduler code | Richard van der Hoff | 2019-08-20 | 1 | -45/+65 |
| | | | | | | | | | | | | | | | | Get rid of the labyrinthine `recoverer_fn` code, and clean up the startup code (it seemed to be previously inexplicably split between `ApplicationServiceScheduler.start` and `_Recoverer.start`). Add some docstrings too. | ||||
* | | Avoid deep recursion in appservice recovery (#5885) | Richard van der Hoff | 2019-08-20 | 1 | -18/+25 |
|/ | | | | | | | | | | Hopefully, this will fix a stack overflow when recovering an appservice. The recursion here leads to a huge chain of deferred callbacks, which then overflows the stack when the chain completes. `inlineCallbacks` makes a better job of this if we use iteration instead. Clean up the code a bit too, while we're there. | ||||
* | Replace returnValue with return (#5736) | Amber Brown | 2019-07-23 | 1 | -2/+2 |
| | |||||
* | Move logging utilities out of the side drawer of util/ and into logging/ (#5606) | Amber Brown | 2019-07-04 | 1 | -1/+1 |
| | |||||
* | Run Black. (#5482) | Amber Brown | 2019-06-20 | 1 | -30/+20 |
| | |||||
* | Run the AS senders as background processes (#4189) | Richard van der Hoff | 2018-12-04 | 1 | -12/+24 |
| | | | | This should fix some "Starting db connection from sentinel context" warnings, and will mean we get metrics for these processes. | ||||
* | run isort | Amber Brown | 2018-07-09 | 1 | -2/+2 |
| | |||||
* | Merge remote-tracking branch 'origin/develop' into rav/use_run_in_background | Richard van der Hoff | 2018-04-27 | 1 | -11/+14 |
|\ | |||||
| * | Improve exception handling for background processes | Richard van der Hoff | 2018-04-27 | 1 | -11/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were a bunch of places where we fire off a process to happen in the background, but don't have any exception handling on it - instead relying on the unhandled error being logged when the relevent deferred gets garbage-collected. This is unsatisfactory for a number of reasons: - logging on garbage collection is best-effort and may happen some time after the error, if at all - it can be hard to figure out where the error actually happened. - it is logged as a scary CRITICAL error which (a) I always forget to grep for and (b) it's not really CRITICAL if a background process we don't care about fails. So this is an attempt to add exception handling to everything we fire off into the background. | ||||
* | | Use run_in_background in preference to preserve_fn | Richard van der Hoff | 2018-04-27 | 1 | -6/+6 |
|/ | | | | | | While I was going through uses of preserve_fn for other PRs, I converted places which only use the wrapped function once to use run_in_background, to avoid creating the function object. | ||||
* | replace 'except:' with 'except Exception:' | Richard van der Hoff | 2017-10-23 | 1 | -1/+1 |
| | | | | what could possibly go wrong | ||||
* | Preserve some logcontexts | Erik Johnston | 2016-08-24 | 1 | -3/+3 |
| | |||||
* | Change name of metric | Matrix | 2016-08-17 | 1 | -1/+1 |
| | |||||
* | Move Measure block inside loop | Erik Johnston | 2016-08-17 | 1 | -9/+9 |
| | |||||
* | Clean up _ServiceQueuer | Erik Johnston | 2016-08-17 | 1 | -31/+30 |
| | |||||
* | Move the AS handler out of the Handlers object. | Mark Haines | 2016-05-31 | 1 | -7/+7 |
| | | | | | | Access it directly from the homeserver itself. It already wasn't inheriting from BaseHandler storing it on the Handlers object was already somewhat dubious. | ||||
* | copyrights | Matthew Hodgson | 2016-01-07 | 1 | -1/+1 |
| | |||||
* | Cap the time to retry txns to appservices to 8.5 minutes | Kegan Dougal | 2015-10-21 | 1 | -2/+2 |
| | | | | | | | | | | | There's been numerous issues with people playing around with their application service and then not receiving events from their HS for ages due to backoff timers reaching crazy heights (albeit capped at < 1 day). Reduce the max time between pokes to be 8.5 minutes (2^9 secs) which is quick enough for people to wait it out (avg wait time being 4.25 min) but long enough to actually give the AS breathing room if it needs it. | ||||
* | Fixes from PR comments | Kegan Dougal | 2015-03-26 | 1 | -1/+2 |
| | |||||
* | Implement ServiceQueuer with tests. | Kegan Dougal | 2015-03-16 | 1 | -16/+30 |
| | |||||
* | Replace EventGrouper for ServiceQueuer to move to push-based txns. Fix tests ↵ | Kegan Dougal | 2015-03-16 | 1 | -28/+33 |
| | | | | and add stub tests for ServiceQueuer. | ||||
* | Wrap polling/retry blocks in try/excepts to avoid sending to other ASes ↵ | Kegan Dougal | 2015-03-16 | 1 | -29/+39 |
| | | | | breaking permanently should an error occur. | ||||
* | Minor PR comment tweaks. | Kegan Dougal | 2015-03-16 | 1 | -2/+2 |
| | |||||
* | Fix remaining scheduler bugs. Add more informative logging. | Kegan Dougal | 2015-03-10 | 1 | -30/+22 |
| | |||||
* | Bug fixes whilst putting it all together | Kegan Dougal | 2015-03-09 | 1 | -1/+3 |
| | |||||
* | Use seconds; start gluing in the AS scheduler into the AS handler. | Kegan Dougal | 2015-03-09 | 1 | -2/+2 |
| | |||||
* | Upper bound the backoff. | Kegan Dougal | 2015-03-06 | 1 | -1/+3 |
| | |||||
* | Apply clarity and docstrings | Kegan Dougal | 2015-03-06 | 1 | -1/+1 |
| | |||||
* | Finish synapse.appservice.scheduler implementation. | Kegan Dougal | 2015-03-06 | 1 | -51/+12 |
| | | | | | With tests to assert behaviour. Not hooked up yet. Stub datastore methods not implemented yet. | ||||
* | Add some loggers | Kegan Dougal | 2015-03-06 | 1 | -0/+10 |
| | |||||
* | Flesh out more stub functions. | Kegan Dougal | 2015-03-06 | 1 | -8/+29 |
| | |||||
* | Add stub ApplicationServiceTransactionStore. Bootstrap Recoverers. Fill in ↵ | Kegan Dougal | 2015-03-06 | 1 | -7/+43 |
| | | | | stub Transaction functions. | ||||
* | Add more tests; fix bugs. | Kegan Dougal | 2015-03-06 | 1 | -2/+2 |
| | |||||
* | Start adding some tests | Kegan Dougal | 2015-03-06 | 1 | -10/+13 |
| | |||||
* | Rejig structure given the appservice_handler already filters the correct ↵ | Kegan Dougal | 2015-03-06 | 1 | -59/+85 |
| | | | | ASes to use. | ||||
* | Minor tweaks | Kegan Dougal | 2015-03-05 | 1 | -7/+9 |
| | |||||
* | Add stub architecture for txn reliability. | Kegan Dougal | 2015-03-05 | 1 | -10/+111 |
| | |||||
* | Add stub scheduler module for txn reliability | Kegan Dougal | 2015-03-05 | 1 | -0/+68 |