{"action":"created","issue":{"url":"https://api.github.com/repos/mperham/dalli/issues/515","labels_url":"https://api.github.com/repos/mperham/dalli/issues/515/labels{/name}","comments_url":"https://api.github.com/repos/mperham/dalli/issues/515/comments","events_url":"https://api.github.com/repos/mperham/dalli/issues/515/events","html_url":"https://github.com/mperham/dalli/pull/515","id":58729995,"number":515,"title":"Connect to unix socket","user":{"login":"r-stu31","id":325471,"avatar_url":"https://avatars.githubusercontent.com/u/325471?v=3","gravatar_id":"","url":"https://api.github.com/users/r-stu31","html_url":"https://github.com/r-stu31","followers_url":"https://api.github.com/users/r-stu31/followers","following_url":"https://api.github.com/users/r-stu31/following{/other_user}","gists_url":"https://api.github.com/users/r-stu31/gists{/gist_id}","starred_url":"https://api.github.com/users/r-stu31/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/r-stu31/subscriptions","organizations_url":"https://api.github.com/users/r-stu31/orgs","repos_url":"https://api.github.com/users/r-stu31/repos","events_url":"https://api.github.com/users/r-stu31/events{/privacy}","received_events_url":"https://api.github.com/users/r-stu31/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-02-24T12:12:10Z","updated_at":"2015-02-25T19:30:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mperham/dalli/pulls/515","html_url":"https://github.com/mperham/dalli/pull/515","diff_url":"https://github.com/mperham/dalli/pull/515.diff","patch_url":"https://github.com/mperham/dalli/pull/515.patch"},"body":"Please review and merge the connect_to_unix_socket branch. It adds support for connecting to memcached using a unix socket.\r\n\r\nThe user must use a full pathname (must begin with a slash character: '/') in place of the \"host:port\" pair argument to Dalli::Client.new method for connecting to a unix socket."},"comment":{"url":"https://api.github.com/repos/mperham/dalli/issues/comments/76037032","html_url":"https://github.com/mperham/dalli/pull/515#issuecomment-76037032","issue_url":"https://api.github.com/repos/mperham/dalli/issues/515","id":76037032,"user":{"login":"petergoldstein","id":421488,"avatar_url":"https://avatars.githubusercontent.com/u/421488?v=3","gravatar_id":"","url":"https://api.github.com/users/petergoldstein","html_url":"https://github.com/petergoldstein","followers_url":"https://api.github.com/users/petergoldstein/followers","following_url":"https://api.github.com/users/petergoldstein/following{/other_user}","gists_url":"https://api.github.com/users/petergoldstein/gists{/gist_id}","starred_url":"https://api.github.com/users/petergoldstein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petergoldstein/subscriptions","organizations_url":"https://api.github.com/users/petergoldstein/orgs","repos_url":"https://api.github.com/users/petergoldstein/repos","events_url":"https://api.github.com/users/petergoldstein/events{/privacy}","received_events_url":"https://api.github.com/users/petergoldstein/received_events","type":"User","site_admin":false},"created_at":"2015-02-25T19:30:47Z","updated_at":"2015-02-25T19:30:47Z","body":"@r-stu31 I'm happy to reopen the discussion.  I think @mperham's point about support is well-taken a a general guideline for features, but at the same time I'm not sure whether I agree that this particular feature shouldn't be supported by Dalli.  \r\n\r\nSome pros in my mind:\r\n\r\n1. Dalli is a driver for memcached, and it seems natural for Dalli to support the network interfaces that memcached supports\r\n2. It is not feasible for a developer to extend Dalli to provide this support - it's too fundamental to the library\r\n\r\nSome cons:\r\n\r\n1. Minimal demand from the userbase - the 2.x branch has been out for a long time, and there hasn't been much clamor for this feature that I'm aware of.\r\n2. As @mperham notes, there's a user complexity burden here - I'd want any PR including this functionality to include suitable documentation that would be clear to a novice user.\r\n\r\nI'm in the middle of a few other things at the moment, but I'll give this a more detailed look this weekend and follow up then."}}