Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Go Frontend #130

Merged
merged 46 commits into from
Nov 26, 2014
Merged

Go Frontend #130

merged 46 commits into from
Nov 26, 2014

Conversation

Hamcha
Copy link
Member

@Hamcha Hamcha commented Oct 7, 2014

Since we're at feature-parity with the Erlang frontend I might as well open the PR.

For the current status on the frontend development, check #124

Some API were changed, I changed the docs accordingly and updated all the examples, please double check them to see if I made any typos (plz eng not my nat language xoxoxo etc.)

Travis is being a bitch, it works fine in Buildbot, @kyleterry pls do sumthin.

@@ -1,620 +0,0 @@
/*
* This is where the magic happens.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RIP Oleg magic

@qpfiffer
Copy link
Member

qpfiffer commented Oct 7, 2014

One question about the go frontend: The erlang one guranteed some thread safety by ensuring that operations only happened one at a time. Does the go version do something similar? How do concurrent requests work?

@kyleterry
Copy link
Member

@qpfiffer

@qpfiffer
Copy link
Member

🐼
🆘 👍
👖

qpfiffer added a commit that referenced this pull request Nov 26, 2014
@qpfiffer qpfiffer merged commit 02ea474 into master Nov 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants