Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add a Twisted Service to synapse.app.homeserver #105

Merged
merged 5 commits into from
Mar 10, 2015
Merged

Conversation

erikjohnston
Copy link
Member

This is primarily to allow us to use twistd to profile synapse.

@erikjohnston erikjohnston changed the title Erikj perf Add a Twisted Service to synapse.app.homeserver Mar 10, 2015
@dbkr
Copy link
Member

dbkr commented Mar 10, 2015

I'd probably just add a separate 'run' method to run it rather than a parameter, unless I'm missing some reason why that isn't possible. Otherwise lgtm.

@dbkr
Copy link
Member

dbkr commented Mar 10, 2015

lgtm

erikjohnston added a commit that referenced this pull request Mar 10, 2015
Add a Twisted Service to synapse.app.homeserver
@erikjohnston erikjohnston merged commit e84fe35 into develop Mar 10, 2015
@erikjohnston erikjohnston deleted the erikj-perf branch April 27, 2015 14:22
babolivier added a commit that referenced this pull request Sep 24, 2021
* Use  the BaseReporter super-class for _WrappedRustReporter. (#10799)

This fixes mypy errors with jaeger-client >= 4.7.0 and should be a no-op
for versions before that.

Co-authored-by: Patrick Cloke <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants