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

backup nodes in subscriptions #238

Closed
wants to merge 1 commit into from
Closed

backup nodes in subscriptions #238

wants to merge 1 commit into from

Conversation

prymitive
Copy link
Contributor

fixes #179

I'm making pull request since I fear that patch floating somewhere in comments will be lost at some point. This way it's easier to review and comment on changes.
Would be nice if we had such feature at some point, but I don't depend on it.

@prymitive
Copy link
Contributor Author

would be nice to have in some not very distant future

@unbit unbit modified the milestones: uWSGI 2.1, Future ideas Feb 9, 2014
@unbit
Copy link
Owner

unbit commented Feb 9, 2014

something to think about for 2.1

@unbit
Copy link
Owner

unbit commented Mar 6, 2014

i have reviewed your patch and i like the approach. I would like to rewrite the uwsgi_send_subscription function api too to take a struct instead of a potentially growing list of parameters.

@prymitive
Copy link
Contributor Author

great, do you need me to refactor this patch or you are going to apply some modified version?

@unbit
Copy link
Owner

unbit commented Mar 6, 2014

i will make some modification but the general logic will remain the same (expecially the 'level' thing)

@unbit
Copy link
Owner

unbit commented Jul 11, 2014

I am working on it for 2.1. I think algorithms will remain the same. But before merging i will implement the possibility to add more algorithms via plugins. (in this way users could implement specific algos with specific usages of weights and backup_levels)

@unbit unbit closed this Jul 12, 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.

backup nodes in subscriptions
2 participants