-
Notifications
You must be signed in to change notification settings - Fork 27
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
Question about PULL PUSH and module overall #2
Comments
I have not looked at the PULL/PUSH mechanism in this code yet but I don't see a way for it to work other than maintain connection identity in the request and response headers to and from the upstream worker.
|
@deepfryed , I agree with you, more over: we ought to pass nginx's PULL socket address if we wish one worker to serve several nginx |
sounds like over-engineering - i'd rather put a load balancer in the middle instead |
Hi guys,
Yes.
For now, all sockets are transient per-request, so there is no problem with that. This will obviously change (it's still very much work in progress) and then you will need to pass identification header that came with the request.
There is
For simple request/response - agreed, but that's not the point of this module. |
@PiotrSikora, it sounds very promising :) Will take an eye on this project
Then there could be bound PULL socket and connected-to-many PUSH socket - simple load balancer with fail tolerance enabled :) - had I got the idea? |
@PiotrSikora re. over-engineering, i meant the single worker serving multiple nginx instances workflow. |
@deepfryed, I'm thinking about multiple workers serving multiple nginx instances, so that fail of any worker or fail of any nginx doesn't break the whole thing. Any nginx's request will PULLed by any alive worker using balancing property of PUSH-PULL chain, and worker will PUSH reply, marked by request handle, to nginx's PULL socket, which address also will be encoded in a request. |
Good day.
First of all, excuse me for asking the question instead of reading sources.
With respect,
Yura
The text was updated successfully, but these errors were encountered: