GitHub PullRequestReviewCommentEvent at 2014-04-28 17:24:08
Repository: elephant-bird
Actor: rangadi
Payload: {"comment":{"url":"https://api.github.com/repos/kevinweil/elephant-bird/pulls/comments/12057090","id":12057090,"diff_hunk":"@@ -95,10 +107,14 @@ public JsonLoader(String optString) {\n       configuredOptions_ = parser_.parse(validOptions_, optsArr);\n     } catch (org.apache.commons.cli.ParseException e) {\n         HelpFormatter formatter = new HelpFormatter();\n-        formatter.printHelp( \"[-nestedLoad] [-inputFormat]\", validOptions_ );\n+        formatter.printHelp( \"[-nestedLoad] [-inputFormat] [-invalidRecord]\", validOptions_ );\n         throw new RuntimeException(e);\n     }\n     isNestedLoadEnabled = configuredOptions_.hasOption(\"nestedLoad\");\n+    isInvalidRecordEnabled = configuredOptions_.hasOption(\"invalidRecord\");\n+    if (isInvalidRecordEnabled) {\n+\t\tretMap = Maps.newHashMap();","path":"pig/src/main/java/com/twitter/elephantbird/pig/load/JsonLoader.java","position":50,"original_position":50,"commit_id":"75f98d9723b4b66d049b1f43ce6ed06e9a4b5b4a","original_commit_id":"75f98d9723b4b66d049b1f43ce6ed06e9a4b5b4a","user":{"login":"rangadi","id":502522,"avatar_url":"https://avatars.githubusercontent.com/u/502522?","gravatar_id":"7ad8f8bbeca0b462aa7bf9ab5a4e096c","url":"https://api.github.com/users/rangadi","html_url":"https://github.com/rangadi","followers_url":"https://api.github.com/users/rangadi/followers","following_url":"https://api.github.com/users/rangadi/following{/other_user}","gists_url":"https://api.github.com/users/rangadi/gists{/gist_id}","starred_url":"https://api.github.com/users/rangadi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rangadi/subscriptions","organizations_url":"https://api.github.com/users/rangadi/orgs","repos_url":"https://api.github.com/users/rangadi/repos","events_url":"https://api.github.com/users/rangadi/events{/privacy}","received_events_url":"https://api.github.com/users/rangadi/received_events","type":"User","site_admin":false},"body":"indentation look way off. 2 spaces is preferred.","created_at":"2014-04-28T17:24:08Z","updated_at":"2014-04-28T17:24:08Z","html_url":"https://github.com/kevinweil/elephant-bird/pull/378#discussion_r12057090","pull_request_url":"https://api.github.com/repos/kevinweil/elephant-bird/pulls/378","_links":{"self":{"href":"https://api.github.com/repos/kevinweil/elephant-bird/pulls/comments/12057090"},"html":{"href":"https://github.com/kevinweil/elephant-bird/pull/378#discussion_r12057090"},"pull_request":{"href":"https://api.github.com/repos/kevinweil/elephant-bird/pulls/378"}}}}