-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
nsqd: make importable #325
Conversation
@kzvezdarov thanks for getting this started. It looks like travis is reporting test failures. Looking at the logs it seems like |
As to your question on the ML, once we've got everything working, etc. we'll squash down the commits. |
The last comit should fix the problem. |
@kzvezdarov thanks, travis is happy 😄 I've gone through the public API that this change will now expose (by running
Let's see how the public API looks after that... |
On it. |
In unexporting the structs, should I also unexport their exported fields? |
Good question, I don't think that's necessary though. |
I can't reproduce the above error and I am not too sure what it means/where it could come from? |
Yea, don't worry about it, looks like a flakey test. I'll poke around separately. |
@kzvezdarov you wanna rebase/squash this down and see tests pass? |
Seems well this time. |
great, thanks for your contribution! |
The changes in this PR split the nsqd application into an executable and a package (called nsqd), in the same manner as nsqlookupd.