Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Pacto Server: replace Goliath with Reel #170

Merged
merged 10 commits into from
Feb 17, 2015
Merged

Conversation

maxlinc
Copy link
Collaborator

@maxlinc maxlinc commented Feb 3, 2015

Pacto server has been based on Goliath, which uses EventMachine. Goliath doesn't seem to be under active development, and I've had a lot of trouble with EventMachine ranging from installation issues to it being difficult to do proper exception handling and avoid deadlocks. In fact, Pacto's own tests had to avoid running Pacto Server under JRuby to avoid issues.

This PR switches the server over to Reel, which powered by Celluloid. The Reel-based server seems to be cleaner, faster, more stable on Windows and JRuby, and easier to safely use with applications containing other concurrent processes.

@maxlinc
Copy link
Collaborator Author

maxlinc commented Feb 3, 2015

Note: I also changed the commands to use the stub and proxy commands proposed in #163. README and docs probably need to be updated before this is good to merge.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.14%) to 91.54% when pulling 6dab01c on maxlinc:reel into 6fc55ab on thoughtworks:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+6.14%) to 91.54% when pulling 6dab01c on maxlinc:reel into 6fc55ab on thoughtworks:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.61%) to 89.01% when pulling 1a8b508 on maxlinc:reel into 6fc55ab on thoughtworks:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.43%) to 88.83% when pulling 1a8b508 on maxlinc:reel into 6fc55ab on thoughtworks:master.

maxlinc added a commit that referenced this pull request Feb 17, 2015
Pacto Server: replace Goliath with Reel
@maxlinc maxlinc merged commit 9964685 into thoughtworks:master Feb 17, 2015
@maxlinc maxlinc deleted the reel branch February 17, 2015 21:12
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.29%) to 84.11% when pulling 0124f71 on maxlinc:reel into 6fc55ab on thoughtworks:master.

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