{"action":"created","issue":{"url":"https://api.github.com/repos/citusdata/pg_shard/issues/57","labels_url":"https://api.github.com/repos/citusdata/pg_shard/issues/57/labels{/name}","comments_url":"https://api.github.com/repos/citusdata/pg_shard/issues/57/comments","events_url":"https://api.github.com/repos/citusdata/pg_shard/issues/57/events","html_url":"https://github.com/citusdata/pg_shard/pull/57","id":54879565,"number":57,"title":"pg_shard performance improvement","user":{"login":"onderkalaci","id":7829470,"avatar_url":"https://avatars.githubusercontent.com/u/7829470?v=3","gravatar_id":"","url":"https://api.github.com/users/onderkalaci","html_url":"https://github.com/onderkalaci","followers_url":"https://api.github.com/users/onderkalaci/followers","following_url":"https://api.github.com/users/onderkalaci/following{/other_user}","gists_url":"https://api.github.com/users/onderkalaci/gists{/gist_id}","starred_url":"https://api.github.com/users/onderkalaci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/onderkalaci/subscriptions","organizations_url":"https://api.github.com/users/onderkalaci/orgs","repos_url":"https://api.github.com/users/onderkalaci/repos","events_url":"https://api.github.com/users/onderkalaci/events{/privacy}","received_events_url":"https://api.github.com/users/onderkalaci/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/citusdata/pg_shard/labels/waffle:%20needs%20review","name":"waffle: needs review","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"jasonmp85","id":50673,"avatar_url":"https://avatars.githubusercontent.com/u/50673?v=3","gravatar_id":"","url":"https://api.github.com/users/jasonmp85","html_url":"https://github.com/jasonmp85","followers_url":"https://api.github.com/users/jasonmp85/followers","following_url":"https://api.github.com/users/jasonmp85/following{/other_user}","gists_url":"https://api.github.com/users/jasonmp85/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonmp85/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonmp85/subscriptions","organizations_url":"https://api.github.com/users/jasonmp85/orgs","repos_url":"https://api.github.com/users/jasonmp85/repos","events_url":"https://api.github.com/users/jasonmp85/events{/privacy}","received_events_url":"https://api.github.com/users/jasonmp85/received_events","type":"User","site_admin":false},"milestone":null,"comments":8,"created_at":"2015-01-20T12:54:18Z","updated_at":"2015-01-23T20:57:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/citusdata/pg_shard/pulls/57","html_url":"https://github.com/citusdata/pg_shard/pull/57","diff_url":"https://github.com/citusdata/pg_shard/pull/57.diff","patch_url":"https://github.com/citusdata/pg_shard/pull/57.patch"},"body":"This branch implements a cache in front of  *GetOperatorByType* function, so that it prevents unnecessary calls to *GetDefaultOpClass* function.\r\n\r\nWith this implementation, we get the same performance improvement that we observed with @ozgune. I may send the details of the results via an email.\r\n\r\nfixes #50\r\n\r\nReview tasks:\r\n  - [x] Replace appearances of `OperatorType` with `OperatorId` (also check comments)\r\n  - [x] Short-circuit without caching if `GetOperatorByType` returns `InvalidOid`\r\n  - [x] Clean up the `if` statement with the three boolean expressions\r\n  - [x] Ensure function declaration order matches definition order\r\n"},"comment":{"url":"https://api.github.com/repos/citusdata/pg_shard/issues/comments/71264520","html_url":"https://github.com/citusdata/pg_shard/pull/57#issuecomment-71264520","issue_url":"https://api.github.com/repos/citusdata/pg_shard/issues/57","id":71264520,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2015-01-23T20:57:44Z","updated_at":"2015-01-23T20:57:44Z","body":"\n[![Coverage Status](https://coveralls.io/builds/1796253/badge)](https://coveralls.io/builds/1796253)\n\nCoverage increased (+0.12%) to 93.2% when pulling **ddd57b647df7384121a93cf4682adb8f1435b036 on feature-issue#50** into **16079449a57a4797624cabb60fe0704004ae29bd on develop**.\n"}}