{"action":"created","issue":{"url":"https://api.github.com/repos/infinispan/infinispan/issues/3648","labels_url":"https://api.github.com/repos/infinispan/infinispan/issues/3648/labels{/name}","comments_url":"https://api.github.com/repos/infinispan/infinispan/issues/3648/comments","events_url":"https://api.github.com/repos/infinispan/infinispan/issues/3648/events","html_url":"https://github.com/infinispan/infinispan/pull/3648","id":100765174,"number":3648,"title":"[ISPN-5517] JDBCBinaryStore size method should query count directly i…","user":{"login":"ryanemerson","id":765332,"avatar_url":"https://avatars.githubusercontent.com/u/765332?v=3","gravatar_id":"","url":"https://api.github.com/users/ryanemerson","html_url":"https://github.com/ryanemerson","followers_url":"https://api.github.com/users/ryanemerson/followers","following_url":"https://api.github.com/users/ryanemerson/following{/other_user}","gists_url":"https://api.github.com/users/ryanemerson/gists{/gist_id}","starred_url":"https://api.github.com/users/ryanemerson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ryanemerson/subscriptions","organizations_url":"https://api.github.com/users/ryanemerson/orgs","repos_url":"https://api.github.com/users/ryanemerson/repos","events_url":"https://api.github.com/users/ryanemerson/events{/privacy}","received_events_url":"https://api.github.com/users/ryanemerson/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/infinispan/infinispan/labels/Question","name":"Question","color":"bfe5bf"},{"url":"https://api.github.com/repos/infinispan/infinispan/labels/Ready%20for%20Review","name":"Ready for Review","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-13T13:08:32Z","updated_at":"2015-08-20T09:01:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/infinispan/infinispan/pulls/3648","html_url":"https://github.com/infinispan/infinispan/pull/3648","diff_url":"https://github.com/infinispan/infinispan/pull/3648.diff","patch_url":"https://github.com/infinispan/infinispan/pull/3648.patch"},"body":"…nstead of iterating"},"comment":{"url":"https://api.github.com/repos/infinispan/infinispan/issues/comments/132944470","html_url":"https://github.com/infinispan/infinispan/pull/3648#issuecomment-132944470","issue_url":"https://api.github.com/repos/infinispan/infinispan/issues/3648","id":132944470,"user":{"login":"ryanemerson","id":765332,"avatar_url":"https://avatars.githubusercontent.com/u/765332?v=3","gravatar_id":"","url":"https://api.github.com/users/ryanemerson","html_url":"https://github.com/ryanemerson","followers_url":"https://api.github.com/users/ryanemerson/followers","following_url":"https://api.github.com/users/ryanemerson/following{/other_user}","gists_url":"https://api.github.com/users/ryanemerson/gists{/gist_id}","starred_url":"https://api.github.com/users/ryanemerson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ryanemerson/subscriptions","organizations_url":"https://api.github.com/users/ryanemerson/orgs","repos_url":"https://api.github.com/users/ryanemerson/repos","events_url":"https://api.github.com/users/ryanemerson/events{/privacy}","received_events_url":"https://api.github.com/users/ryanemerson/received_events","type":"User","site_admin":false},"created_at":"2015-08-20T09:01:13Z","updated_at":"2015-08-20T09:01:13Z","body":"@galderz Results of a single execution.  \r\nWhen a cache contains 240 entries, none of which have expired, the old method takes 7950521ns whereas the new takes 765363ns. \r\n\r\nOnce we have moved time forward and half of the entries in the cache have expired, the old method takes 579729ns and the new method 184852ns. \r\n\r\nThroughout testing I never encountered an instance where the old implementation outperformed the new one, however my test case is hardly exhaustive.  "}}