{"action":"created","issue":{"url":"https://api.github.com/repos/google/cayley/issues/229","labels_url":"https://api.github.com/repos/google/cayley/issues/229/labels{/name}","comments_url":"https://api.github.com/repos/google/cayley/issues/229/comments","events_url":"https://api.github.com/repos/google/cayley/issues/229/events","html_url":"https://github.com/google/cayley/pull/229","id":68530997,"number":229,"title":"Add Err() and Close() methods to Iterators","user":{"login":"andrew-d","id":1079173,"avatar_url":"https://avatars.githubusercontent.com/u/1079173?v=3","gravatar_id":"","url":"https://api.github.com/users/andrew-d","html_url":"https://github.com/andrew-d","followers_url":"https://api.github.com/users/andrew-d/followers","following_url":"https://api.github.com/users/andrew-d/following{/other_user}","gists_url":"https://api.github.com/users/andrew-d/gists{/gist_id}","starred_url":"https://api.github.com/users/andrew-d/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrew-d/subscriptions","organizations_url":"https://api.github.com/users/andrew-d/orgs","repos_url":"https://api.github.com/users/andrew-d/repos","events_url":"https://api.github.com/users/andrew-d/events{/privacy}","received_events_url":"https://api.github.com/users/andrew-d/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":24,"created_at":"2015-04-15T00:00:18Z","updated_at":"2015-04-16T18:58:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/google/cayley/pulls/229","html_url":"https://github.com/google/cayley/pull/229","diff_url":"https://github.com/google/cayley/pull/229.diff","patch_url":"https://github.com/google/cayley/pull/229.patch"},"body":"This PR does the following:\r\n- As mentioned in #208, add an `Err()` method to all Iterators.\r\n- Adds tests that the various iterators pass through the underlying Err value.\r\n- Makes the `Close()` method on Iterators return an error.\r\n- Remove a bunch of uses of `glog.Fatal*` calls, instead either propagating the errors, or calling `panic` (when it's a programming bug).\r\n- Try to handle errors in other locations that use `Next()`.\r\n\r\ncc @kortschak \r\n\r\nUpdates #157 "},"comment":{"url":"https://api.github.com/repos/google/cayley/issues/comments/93815244","html_url":"https://github.com/google/cayley/pull/229#issuecomment-93815244","issue_url":"https://api.github.com/repos/google/cayley/issues/229","id":93815244,"user":{"login":"barakmich","id":98356,"avatar_url":"https://avatars.githubusercontent.com/u/98356?v=3","gravatar_id":"","url":"https://api.github.com/users/barakmich","html_url":"https://github.com/barakmich","followers_url":"https://api.github.com/users/barakmich/followers","following_url":"https://api.github.com/users/barakmich/following{/other_user}","gists_url":"https://api.github.com/users/barakmich/gists{/gist_id}","starred_url":"https://api.github.com/users/barakmich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/barakmich/subscriptions","organizations_url":"https://api.github.com/users/barakmich/orgs","repos_url":"https://api.github.com/users/barakmich/repos","events_url":"https://api.github.com/users/barakmich/events{/privacy}","received_events_url":"https://api.github.com/users/barakmich/received_events","type":"User","site_admin":false},"created_at":"2015-04-16T18:58:43Z","updated_at":"2015-04-16T18:58:43Z","body":"Just cause I've gotta ask: You've signed the CLA?"}}