The entire world uses : to separate host from port, why did memcache acm backend choose to use / for this purpose?
http://tps.projects.bsdpower.com/changeset/10579
/ as host-port separator in memcache multi server commit
Forum rules
Discussion of general topics related to the new release and its place in the world. Don't discuss new features, report bugs, ask for support, et cetera. Don't use this to spam for other boards or attack those boards!
Discussion of general topics related to the new release and its place in the world. Don't discuss new features, report bugs, ask for support, et cetera. Don't use this to spam for other boards or attack those boards!
Re: / as host-port separator in memcache multi server commit
Quite simply IPv6.
Re: / as host-port separator in memcache multi server commit
Wouldn't it be better to allow host-port separator to be configurable, perhaps via another constant, and default it to colon?
This may be a bike shed discussion, but then who accesses memcached servers via ipv6?
Also, documentation for memcached constants in docs/coding-guidelines.html should be updated.
This may be a bike shed discussion, but then who accesses memcached servers via ipv6?
Also, documentation for memcached constants in docs/coding-guidelines.html should be updated.
Re: / as host-port separator in memcache multi server commit
The idea was to keep it simple so we wouldn't have to do too much parsing, so that seems like a very unecessary overcomplication.nn- wrote:Wouldn't it be better to allow host-port separator to be configurable, perhaps via another constant, and default it to colon?
That is actually a whole lot more likely than a webserver accessible via ipv6 right now since IPv6 is already being used for internal network connections in some places. However "who needs multiple memcached servers at all?" seems like a reasonable question and the answer is very few. For everyone else there is a separate HOST/PORT constant anyway.nn- wrote:This may be a bike shed discussion, but then who accesses memcached servers via ipv6?
Indeed.nn- wrote:Also, documentation for memcached constants in docs/coding-guidelines.html should be updated.