We have added this so as to do Round robin among the upstreams running the same version as the version of the upstream where the client's mail account resides (which i agree is again specific to us). Although i didn't include our changes in ngx_http_upstream_round_robin.c (as its based on our implementation of how the 'version' string looks like), i thought atleast the generic change of supporting 'version' in server directive can be used upstream by nginx too ?
Thanks -Kunal ----- Original Message ----- From: "Maxim Dounin" <mdou...@mdounin.ru> To: "nginx-devel" <nginx-devel@nginx.org> Sent: Thursday, September 18, 2014 4:00:54 AM Subject: Re: [PATCH] Add 'version' parameter for http upstream server directive Hello! On Wed, Sep 17, 2014 at 05:59:23PM -0500, Kunal Pariani wrote: > # HG changeset patch > # User Kunal Pariani <kpari...@zimbra.com> > # Date 1410994128 18000 > # Wed Sep 17 17:48:48 2014 -0500 > # Node ID 1cad46e7189fc43d50a24735f61793f710f78967 > # Parent dfc11aa2a98e697a562b2aee092a77e3dc13f55a > Add 'version' parameter for http upstream server directive http://nginx.org/en/docs/contributing_changes.html "Try to make it clear why the suggested change is needed, and provide a use case, if possible." -- Maxim Dounin http://nginx.org/ _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel
