{"action":"created","comment":{"url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/comments/124087898","pull_request_review_id":46347183,"id":124087898,"diff_hunk":"@@ -462,6 +472,12 @@ def search(self, query_string, sort_by=None, start_offset=0, end_offset=None,\n             self.log.error(\"Failed to query Solr using '%s': %s\", query_string, e)\n             raw_results = EmptyResults()\n \n+        if requested_fields_in_redis:","path":"seo/search_backend.py","position":80,"original_position":80,"commit_id":"04f10096d58d996680cccf6b50e2dbd86386788f","original_commit_id":"04f10096d58d996680cccf6b50e2dbd86386788f","user":{"login":"almathew","id":2728248,"avatar_url":"https://avatars1.githubusercontent.com/u/2728248?v=3","gravatar_id":"","url":"https://api.github.com/users/almathew","html_url":"https://github.com/almathew","followers_url":"https://api.github.com/users/almathew/followers","following_url":"https://api.github.com/users/almathew/following{/other_user}","gists_url":"https://api.github.com/users/almathew/gists{/gist_id}","starred_url":"https://api.github.com/users/almathew/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/almathew/subscriptions","organizations_url":"https://api.github.com/users/almathew/orgs","repos_url":"https://api.github.com/users/almathew/repos","events_url":"https://api.github.com/users/almathew/events{/privacy}","received_events_url":"https://api.github.com/users/almathew/received_events","type":"User","site_admin":false},"body":"Sure, that's exactly what I was hoping for.","created_at":"2017-06-26T18:43:04Z","updated_at":"2017-06-26T18:43:04Z","html_url":"https://github.com/DirectEmployers/MyJobs/pull/3169#discussion_r124087898","pull_request_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169","_links":{"self":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/comments/124087898"},"html":{"href":"https://github.com/DirectEmployers/MyJobs/pull/3169#discussion_r124087898"},"pull_request":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169"}}},"pull_request":{"url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169","id":127304115,"html_url":"https://github.com/DirectEmployers/MyJobs/pull/3169","diff_url":"https://github.com/DirectEmployers/MyJobs/pull/3169.diff","patch_url":"https://github.com/DirectEmployers/MyJobs/pull/3169.patch","issue_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/3169","number":3169,"state":"open","locked":false,"title":"Pd 3363 get guids from solr and data from Redis","user":{"login":"vigowei","id":27963853,"avatar_url":"https://avatars0.githubusercontent.com/u/27963853?v=3","gravatar_id":"","url":"https://api.github.com/users/vigowei","html_url":"https://github.com/vigowei","followers_url":"https://api.github.com/users/vigowei/followers","following_url":"https://api.github.com/users/vigowei/following{/other_user}","gists_url":"https://api.github.com/users/vigowei/gists{/gist_id}","starred_url":"https://api.github.com/users/vigowei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vigowei/subscriptions","organizations_url":"https://api.github.com/users/vigowei/orgs","repos_url":"https://api.github.com/users/vigowei/repos","events_url":"https://api.github.com/users/vigowei/events{/privacy}","received_events_url":"https://api.github.com/users/vigowei/received_events","type":"User","site_admin":false},"body":"## What's this PR do?\r\nAdding logic to get guids from solr first then pull more information from Redis or MySQL if needed. \r\n\r\n## Screenshots (if appropriate)\r\nN/A\r\n\r\n## Where should the reviewer start?\r\nseo/search_backend.py line#302\r\n\r\n## How should this be manually tested?\r\n job search on microsites should still work,\r\n\r\n## Any background context you want to provide?\r\nTo have it pull data from outside of solr, add fields to SEO_REDIS_FIELDS.\r\nStill fixing some tests.\r\n","created_at":"2017-06-23T23:18:56Z","updated_at":"2017-06-26T18:43:04Z","closed_at":null,"merged_at":null,"merge_commit_sha":"ed1547e65d7e2c8500a577d28f32ed2aa8528c8c","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169/commits","review_comments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169/comments","review_comment_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/comments{/number}","comments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/3169/comments","statuses_url":"https://api.github.com/repos/DirectEmployers/MyJobs/statuses/04f10096d58d996680cccf6b50e2dbd86386788f","head":{"label":"DirectEmployers:PD-3363-get-guids-from-solr","ref":"PD-3363-get-guids-from-solr","sha":"04f10096d58d996680cccf6b50e2dbd86386788f","user":{"login":"DirectEmployers","id":381171,"avatar_url":"https://avatars1.githubusercontent.com/u/381171?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectEmployers","html_url":"https://github.com/DirectEmployers","followers_url":"https://api.github.com/users/DirectEmployers/followers","following_url":"https://api.github.com/users/DirectEmployers/following{/other_user}","gists_url":"https://api.github.com/users/DirectEmployers/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectEmployers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectEmployers/subscriptions","organizations_url":"https://api.github.com/users/DirectEmployers/orgs","repos_url":"https://api.github.com/users/DirectEmployers/repos","events_url":"https://api.github.com/users/DirectEmployers/events{/privacy}","received_events_url":"https://api.github.com/users/DirectEmployers/received_events","type":"Organization","site_admin":false},"repo":{"id":2867982,"name":"MyJobs","full_name":"DirectEmployers/MyJobs","owner":{"login":"DirectEmployers","id":381171,"avatar_url":"https://avatars1.githubusercontent.com/u/381171?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectEmployers","html_url":"https://github.com/DirectEmployers","followers_url":"https://api.github.com/users/DirectEmployers/followers","following_url":"https://api.github.com/users/DirectEmployers/following{/other_user}","gists_url":"https://api.github.com/users/DirectEmployers/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectEmployers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectEmployers/subscriptions","organizations_url":"https://api.github.com/users/DirectEmployers/orgs","repos_url":"https://api.github.com/users/DirectEmployers/repos","events_url":"https://api.github.com/users/DirectEmployers/events{/privacy}","received_events_url":"https://api.github.com/users/DirectEmployers/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/DirectEmployers/MyJobs","description":"Multi-user website designed to assist job seekers in their search for employment","fork":false,"url":"https://api.github.com/repos/DirectEmployers/MyJobs","forks_url":"https://api.github.com/repos/DirectEmployers/MyJobs/forks","keys_url":"https://api.github.com/repos/DirectEmployers/MyJobs/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DirectEmployers/MyJobs/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DirectEmployers/MyJobs/teams","hooks_url":"https://api.github.com/repos/DirectEmployers/MyJobs/hooks","issue_events_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/events{/number}","events_url":"https://api.github.com/repos/DirectEmployers/MyJobs/events","assignees_url":"https://api.github.com/repos/DirectEmployers/MyJobs/assignees{/user}","branches_url":"https://api.github.com/repos/DirectEmployers/MyJobs/branches{/branch}","tags_url":"https://api.github.com/repos/DirectEmployers/MyJobs/tags","blobs_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/refs{/sha}","trees_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DirectEmployers/MyJobs/statuses/{sha}","languages_url":"https://api.github.com/repos/DirectEmployers/MyJobs/languages","stargazers_url":"https://api.github.com/repos/DirectEmployers/MyJobs/stargazers","contributors_url":"https://api.github.com/repos/DirectEmployers/MyJobs/contributors","subscribers_url":"https://api.github.com/repos/DirectEmployers/MyJobs/subscribers","subscription_url":"https://api.github.com/repos/DirectEmployers/MyJobs/subscription","commits_url":"https://api.github.com/repos/DirectEmployers/MyJobs/commits{/sha}","git_commits_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/commits{/sha}","comments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/comments{/number}","issue_comment_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/comments{/number}","contents_url":"https://api.github.com/repos/DirectEmployers/MyJobs/contents/{+path}","compare_url":"https://api.github.com/repos/DirectEmployers/MyJobs/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DirectEmployers/MyJobs/merges","archive_url":"https://api.github.com/repos/DirectEmployers/MyJobs/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DirectEmployers/MyJobs/downloads","issues_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues{/number}","pulls_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls{/number}","milestones_url":"https://api.github.com/repos/DirectEmployers/MyJobs/milestones{/number}","notifications_url":"https://api.github.com/repos/DirectEmployers/MyJobs/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DirectEmployers/MyJobs/labels{/name}","releases_url":"https://api.github.com/repos/DirectEmployers/MyJobs/releases{/id}","deployments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/deployments","created_at":"2011-11-28T15:22:36Z","updated_at":"2017-05-02T15:55:53Z","pushed_at":"2017-06-26T18:23:53Z","git_url":"git://github.com/DirectEmployers/MyJobs.git","ssh_url":"git@github.com:DirectEmployers/MyJobs.git","clone_url":"https://github.com/DirectEmployers/MyJobs.git","svn_url":"https://github.com/DirectEmployers/MyJobs","homepage":"http://my.jobs","size":38663,"stargazers_count":15,"watchers_count":15,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":12,"mirror_url":null,"open_issues_count":10,"forks":12,"open_issues":10,"watchers":15,"default_branch":"quality-control"}},"base":{"label":"DirectEmployers:solr-improvements","ref":"solr-improvements","sha":"7b7088d94dbcb6533819266f3860131062511a70","user":{"login":"DirectEmployers","id":381171,"avatar_url":"https://avatars1.githubusercontent.com/u/381171?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectEmployers","html_url":"https://github.com/DirectEmployers","followers_url":"https://api.github.com/users/DirectEmployers/followers","following_url":"https://api.github.com/users/DirectEmployers/following{/other_user}","gists_url":"https://api.github.com/users/DirectEmployers/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectEmployers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectEmployers/subscriptions","organizations_url":"https://api.github.com/users/DirectEmployers/orgs","repos_url":"https://api.github.com/users/DirectEmployers/repos","events_url":"https://api.github.com/users/DirectEmployers/events{/privacy}","received_events_url":"https://api.github.com/users/DirectEmployers/received_events","type":"Organization","site_admin":false},"repo":{"id":2867982,"name":"MyJobs","full_name":"DirectEmployers/MyJobs","owner":{"login":"DirectEmployers","id":381171,"avatar_url":"https://avatars1.githubusercontent.com/u/381171?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectEmployers","html_url":"https://github.com/DirectEmployers","followers_url":"https://api.github.com/users/DirectEmployers/followers","following_url":"https://api.github.com/users/DirectEmployers/following{/other_user}","gists_url":"https://api.github.com/users/DirectEmployers/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectEmployers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectEmployers/subscriptions","organizations_url":"https://api.github.com/users/DirectEmployers/orgs","repos_url":"https://api.github.com/users/DirectEmployers/repos","events_url":"https://api.github.com/users/DirectEmployers/events{/privacy}","received_events_url":"https://api.github.com/users/DirectEmployers/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/DirectEmployers/MyJobs","description":"Multi-user website designed to assist job seekers in their search for employment","fork":false,"url":"https://api.github.com/repos/DirectEmployers/MyJobs","forks_url":"https://api.github.com/repos/DirectEmployers/MyJobs/forks","keys_url":"https://api.github.com/repos/DirectEmployers/MyJobs/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DirectEmployers/MyJobs/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DirectEmployers/MyJobs/teams","hooks_url":"https://api.github.com/repos/DirectEmployers/MyJobs/hooks","issue_events_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/events{/number}","events_url":"https://api.github.com/repos/DirectEmployers/MyJobs/events","assignees_url":"https://api.github.com/repos/DirectEmployers/MyJobs/assignees{/user}","branches_url":"https://api.github.com/repos/DirectEmployers/MyJobs/branches{/branch}","tags_url":"https://api.github.com/repos/DirectEmployers/MyJobs/tags","blobs_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/refs{/sha}","trees_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DirectEmployers/MyJobs/statuses/{sha}","languages_url":"https://api.github.com/repos/DirectEmployers/MyJobs/languages","stargazers_url":"https://api.github.com/repos/DirectEmployers/MyJobs/stargazers","contributors_url":"https://api.github.com/repos/DirectEmployers/MyJobs/contributors","subscribers_url":"https://api.github.com/repos/DirectEmployers/MyJobs/subscribers","subscription_url":"https://api.github.com/repos/DirectEmployers/MyJobs/subscription","commits_url":"https://api.github.com/repos/DirectEmployers/MyJobs/commits{/sha}","git_commits_url":"https://api.github.com/repos/DirectEmployers/MyJobs/git/commits{/sha}","comments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/comments{/number}","issue_comment_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/comments{/number}","contents_url":"https://api.github.com/repos/DirectEmployers/MyJobs/contents/{+path}","compare_url":"https://api.github.com/repos/DirectEmployers/MyJobs/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DirectEmployers/MyJobs/merges","archive_url":"https://api.github.com/repos/DirectEmployers/MyJobs/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DirectEmployers/MyJobs/downloads","issues_url":"https://api.github.com/repos/DirectEmployers/MyJobs/issues{/number}","pulls_url":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls{/number}","milestones_url":"https://api.github.com/repos/DirectEmployers/MyJobs/milestones{/number}","notifications_url":"https://api.github.com/repos/DirectEmployers/MyJobs/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DirectEmployers/MyJobs/labels{/name}","releases_url":"https://api.github.com/repos/DirectEmployers/MyJobs/releases{/id}","deployments_url":"https://api.github.com/repos/DirectEmployers/MyJobs/deployments","created_at":"2011-11-28T15:22:36Z","updated_at":"2017-05-02T15:55:53Z","pushed_at":"2017-06-26T18:23:53Z","git_url":"git://github.com/DirectEmployers/MyJobs.git","ssh_url":"git@github.com:DirectEmployers/MyJobs.git","clone_url":"https://github.com/DirectEmployers/MyJobs.git","svn_url":"https://github.com/DirectEmployers/MyJobs","homepage":"http://my.jobs","size":38663,"stargazers_count":15,"watchers_count":15,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":12,"mirror_url":null,"open_issues_count":10,"forks":12,"open_issues":10,"watchers":15,"default_branch":"quality-control"}},"_links":{"self":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169"},"html":{"href":"https://github.com/DirectEmployers/MyJobs/pull/3169"},"issue":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/3169"},"comments":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/issues/3169/comments"},"review_comments":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169/comments"},"review_comment":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/pulls/3169/commits"},"statuses":{"href":"https://api.github.com/repos/DirectEmployers/MyJobs/statuses/04f10096d58d996680cccf6b50e2dbd86386788f"}}}}
{"actor":{"display_login":"almathew"}}