{"action":"created","issue":{"url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/48","labels_url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/48/labels{/name}","comments_url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/48/comments","events_url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/48/events","html_url":"https://github.com/percolatestudio/publish-counts/pull/48","id":97768512,"number":48,"title":"Fix/pr 40 - rewrite cursor field optimization","user":{"login":"boxofrox","id":636691,"avatar_url":"https://avatars.githubusercontent.com/u/636691?v=3","gravatar_id":"","url":"https://api.github.com/users/boxofrox","html_url":"https://github.com/boxofrox","followers_url":"https://api.github.com/users/boxofrox/followers","following_url":"https://api.github.com/users/boxofrox/following{/other_user}","gists_url":"https://api.github.com/users/boxofrox/gists{/gist_id}","starred_url":"https://api.github.com/users/boxofrox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boxofrox/subscriptions","organizations_url":"https://api.github.com/users/boxofrox/orgs","repos_url":"https://api.github.com/users/boxofrox/repos","events_url":"https://api.github.com/users/boxofrox/events{/privacy}","received_events_url":"https://api.github.com/users/boxofrox/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-28T18:21:33Z","updated_at":"2015-07-28T18:26:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/percolatestudio/publish-counts/pulls/48","html_url":"https://github.com/percolatestudio/publish-counts/pull/48","diff_url":"https://github.com/percolatestudio/publish-counts/pull/48.diff","patch_url":"https://github.com/percolatestudio/publish-counts/pull/48.patch"},"body":"Implement the following strategy with test units.\r\n\r\n1. If the user manually provides a fields specifier in their query, we respect that.\r\n\r\n  1. Except for basic counts where the fields are unused and inaccessible, instead warn the user and limit to _id.\r\n\r\n  2. When countFromField* is a string, warn user of potential unused fields, but respect their settings.\r\n\r\n2. If not, and they pass a function in for one of those arguments, we warn them and don't limit the query.\r\n\r\n3. In the usual case, where countFromField* is a string or not set at all, we use what we know to limit fields, and everything is happy."},"comment":{"url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/comments/125704083","html_url":"https://github.com/percolatestudio/publish-counts/pull/48#issuecomment-125704083","issue_url":"https://api.github.com/repos/percolatestudio/publish-counts/issues/48","id":125704083,"user":{"login":"boxofrox","id":636691,"avatar_url":"https://avatars.githubusercontent.com/u/636691?v=3","gravatar_id":"","url":"https://api.github.com/users/boxofrox","html_url":"https://github.com/boxofrox","followers_url":"https://api.github.com/users/boxofrox/followers","following_url":"https://api.github.com/users/boxofrox/following{/other_user}","gists_url":"https://api.github.com/users/boxofrox/gists{/gist_id}","starred_url":"https://api.github.com/users/boxofrox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/boxofrox/subscriptions","organizations_url":"https://api.github.com/users/boxofrox/orgs","repos_url":"https://api.github.com/users/boxofrox/repos","events_url":"https://api.github.com/users/boxofrox/events{/privacy}","received_events_url":"https://api.github.com/users/boxofrox/received_events","type":"User","site_admin":false},"created_at":"2015-07-28T18:26:07Z","updated_at":"2015-07-28T18:26:07Z","body":"Rule 1.ii may be unnecessary.  #47 uses query transforms and would have justification to specify their own query fields, and the warning would not apply.  So 1.ii's main use is hand-holding to remind the user their query field specifier might hinder optimizations."}}