{"action":"created","issue":{"url":"https://api.github.com/repos/hooopo/second_level_cache/issues/68","repository_url":"https://api.github.com/repos/hooopo/second_level_cache","labels_url":"https://api.github.com/repos/hooopo/second_level_cache/issues/68/labels{/name}","comments_url":"https://api.github.com/repos/hooopo/second_level_cache/issues/68/comments","events_url":"https://api.github.com/repos/hooopo/second_level_cache/issues/68/events","html_url":"https://github.com/hooopo/second_level_cache/pull/68","id":222947518,"number":68,"title":"增加 prelaod 对非 integer 的主键类型的支持","user":{"login":"wppurking","id":448364,"avatar_url":"https://avatars3.githubusercontent.com/u/448364?v=3","gravatar_id":"","url":"https://api.github.com/users/wppurking","html_url":"https://github.com/wppurking","followers_url":"https://api.github.com/users/wppurking/followers","following_url":"https://api.github.com/users/wppurking/following{/other_user}","gists_url":"https://api.github.com/users/wppurking/gists{/gist_id}","starred_url":"https://api.github.com/users/wppurking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wppurking/subscriptions","organizations_url":"https://api.github.com/users/wppurking/orgs","repos_url":"https://api.github.com/users/wppurking/repos","events_url":"https://api.github.com/users/wppurking/events{/privacy}","received_events_url":"https://api.github.com/users/wppurking/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-04-20T04:40:16Z","updated_at":"2017-04-20T05:06:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hooopo/second_level_cache/pulls/68","html_url":"https://github.com/hooopo/second_level_cache/pull/68","diff_url":"https://github.com/hooopo/second_level_cache/pull/68.diff","patch_url":"https://github.com/hooopo/second_level_cache/pull/68.patch"},"body":"在使用 uuid 为主键的情况下, 当调用 `Topic.includes(:posts)` 时, preload.rb 中是认为主键一定为 `integer` 类型的. 所以当主键全部为 uuid 的时候是不支持的(pg)\r\n\r\n代码中保留了原来的逻辑, 并添加了针对主键为非 integer 时候的处理. \r\n\r\nsqlite3 的场景已经在 test 中体现了, 只能使用 string 类型.\r\npg 的场景已经在我自己的产品环境中测试了,通过.\r\nmysql 的场景没有真实测试过."},"comment":{"url":"https://api.github.com/repos/hooopo/second_level_cache/issues/comments/295582155","html_url":"https://github.com/hooopo/second_level_cache/pull/68#issuecomment-295582155","issue_url":"https://api.github.com/repos/hooopo/second_level_cache/issues/68","id":295582155,"user":{"login":"wppurking","id":448364,"avatar_url":"https://avatars3.githubusercontent.com/u/448364?v=3","gravatar_id":"","url":"https://api.github.com/users/wppurking","html_url":"https://github.com/wppurking","followers_url":"https://api.github.com/users/wppurking/followers","following_url":"https://api.github.com/users/wppurking/following{/other_user}","gists_url":"https://api.github.com/users/wppurking/gists{/gist_id}","starred_url":"https://api.github.com/users/wppurking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wppurking/subscriptions","organizations_url":"https://api.github.com/users/wppurking/orgs","repos_url":"https://api.github.com/users/wppurking/repos","events_url":"https://api.github.com/users/wppurking/events{/privacy}","received_events_url":"https://api.github.com/users/wppurking/received_events","type":"User","site_admin":false},"created_at":"2017-04-20T05:06:06Z","updated_at":"2017-04-20T05:06:06Z","body":"ci 中没跑过的部分, rubocop: https://github.com/bbatsov/rubocop/pull/4237"}}
{"actor":{"display_login":"wppurking"}}