{"action":"created","issue":{"url":"https://api.github.com/repos/pilosa/pilosa/issues/1167","repository_url":"https://api.github.com/repos/pilosa/pilosa","labels_url":"https://api.github.com/repos/pilosa/pilosa/issues/1167/labels{/name}","comments_url":"https://api.github.com/repos/pilosa/pilosa/issues/1167/comments","events_url":"https://api.github.com/repos/pilosa/pilosa/issues/1167/events","html_url":"https://github.com/pilosa/pilosa/pull/1167","id":305722275,"number":1167,"title":"roaring: speed-up intersectionCountArrayBitmap","user":{"login":"TocarIP","id":16836375,"avatar_url":"https://avatars3.githubusercontent.com/u/16836375?v=4","gravatar_id":"","url":"https://api.github.com/users/TocarIP","html_url":"https://github.com/TocarIP","followers_url":"https://api.github.com/users/TocarIP/followers","following_url":"https://api.github.com/users/TocarIP/following{/other_user}","gists_url":"https://api.github.com/users/TocarIP/gists{/gist_id}","starred_url":"https://api.github.com/users/TocarIP/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TocarIP/subscriptions","organizations_url":"https://api.github.com/users/TocarIP/orgs","repos_url":"https://api.github.com/users/TocarIP/repos","events_url":"https://api.github.com/users/TocarIP/events{/privacy}","received_events_url":"https://api.github.com/users/TocarIP/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-03-15T21:15:28Z","updated_at":"2018-03-22T19:53:27Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/pilosa/pilosa/pulls/1167","html_url":"https://github.com/pilosa/pilosa/pull/1167","diff_url":"https://github.com/pilosa/pilosa/pull/1167.diff","patch_url":"https://github.com/pilosa/pilosa/pull/1167.patch"},"body":"While looking at benchmarks as a possible go compiler benchmarks,\r\nI've tried some optimizations by hand:\r\nMove len(b.bitmap) load out of the loop.\r\nRemove some type conversions.\r\nUse (x >> off) & 1  to get offs bit, instead of  x & (1 << of)f >> off.\r\n\r\nThis produces nice speed-up and passes go test roaring.\r\n\r\nname                                    old time/op  new time/op  delta\r\nBitmap_IntersectionCount_ArrayRun-6     2.06µs ± 0%  1.57µs ± 0%  -24.04%  (p=0.000 n=10+9)\r\nBitmap_IntersectionCount_BitmapRun-6    2.24µs ± 0%  2.24µs ± 0%     ~     (p=0.913 n=10+9)\r\nBitmap_IntersectionCount_ArrayBitmap-6  2.06µs ± 0%  1.56µs ± 1%  -24.05%  (p=0.000 n=9+10)"},"comment":{"url":"https://api.github.com/repos/pilosa/pilosa/issues/comments/375436480","html_url":"https://github.com/pilosa/pilosa/pull/1167#issuecomment-375436480","issue_url":"https://api.github.com/repos/pilosa/pilosa/issues/1167","id":375436480,"user":{"login":"TocarIP","id":16836375,"avatar_url":"https://avatars3.githubusercontent.com/u/16836375?v=4","gravatar_id":"","url":"https://api.github.com/users/TocarIP","html_url":"https://github.com/TocarIP","followers_url":"https://api.github.com/users/TocarIP/followers","following_url":"https://api.github.com/users/TocarIP/following{/other_user}","gists_url":"https://api.github.com/users/TocarIP/gists{/gist_id}","starred_url":"https://api.github.com/users/TocarIP/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TocarIP/subscriptions","organizations_url":"https://api.github.com/users/TocarIP/orgs","repos_url":"https://api.github.com/users/TocarIP/repos","events_url":"https://api.github.com/users/TocarIP/events{/privacy}","received_events_url":"https://api.github.com/users/TocarIP/received_events","type":"User","site_admin":false},"created_at":"2018-03-22T19:53:27Z","updated_at":"2018-03-22T19:53:27Z","author_association":"NONE","body":"I've signed CLA, so can someone merge this?"}}
{"actor":{"display_login":"TocarIP"}}