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

feature: support detective 'zone' #67

Closed
wants to merge 8 commits into from

Conversation

chronolaw
Copy link
Contributor

When we use detective zone in upstream, Nginx will copy all peers into shared memory, and change their storage from array(peers[n]) to list(peer->next). So API get_peers and set_peer_down will fail or case fatal error.

I modified some code to enable this module to work with zone, and mimic some test cases.

Please check these code, thanks.

@chronolaw
Copy link
Contributor Author

Sorry, I just saw #65 , we are talking the same question.
In my implement I also added rwlock and fixed set_peer_down, more code and test cases compared to #65.

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.

1 participant