GitHub PullRequestReviewCommentEvent at 2014-02-19 07:27:33
Repository: presto
Actor: zhenxiao
Payload: {"comment":{"url":"https://api.github.com/repos/facebook/presto/pulls/comments/9856298","id":9856298,"diff_hunk":"@@ -149,14 +182,33 @@ public LocatedFileStatus next()\n     public FileStatus getFileStatus(Path path)\n             throws IOException\n     {\n-        ObjectMetadata metadata = getS3ObjectMetadata(path);\n-        return new FileStatus(\n-                metadata.getContentLength(),\n+        try {\n+            ObjectMetadata metadata = getS3ObjectMetadata(path);\n+            if (metadata == null) {\n+                throw new FileNotFoundException(\"Path \" + path.toString() + \" not found\");","path":"presto-hive/src/main/java/com/facebook/presto/hive/PrestoS3FileSystem.java","position":92,"original_position":92,"commit_id":"8a7213a486e76b14a4b02423b83d3cb9e6a2d9b1","original_commit_id":"8a7213a486e76b14a4b02423b83d3cb9e6a2d9b1","user":{"login":"zhenxiao","id":1274155,"avatar_url":"https://gravatar.com/avatar/d81837bc3fa2b1d4027e2226c5172985?d=https%3A%2F%2Fidenticons.github.com%2F9a5c61fcd0ae03d011cf67dd92416899.png&r=x","gravatar_id":"d81837bc3fa2b1d4027e2226c5172985","url":"https://api.github.com/users/zhenxiao","html_url":"https://github.com/zhenxiao","followers_url":"https://api.github.com/users/zhenxiao/followers","following_url":"https://api.github.com/users/zhenxiao/following{/other_user}","gists_url":"https://api.github.com/users/zhenxiao/gists{/gist_id}","starred_url":"https://api.github.com/users/zhenxiao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zhenxiao/subscriptions","organizations_url":"https://api.github.com/users/zhenxiao/orgs","repos_url":"https://api.github.com/users/zhenxiao/repos","events_url":"https://api.github.com/users/zhenxiao/events{/privacy}","received_events_url":"https://api.github.com/users/zhenxiao/received_events","type":"User","site_admin":false},"body":"@electrum Thanks, I will fix.","created_at":"2014-02-19T07:27:33Z","updated_at":"2014-02-19T07:27:33Z","html_url":"https://github.com/facebook/presto/pull/1044#discussion_r9856298","pull_request_url":"https://api.github.com/repos/facebook/presto/pulls/1044","_links":{"self":{"href":"https://api.github.com/repos/facebook/presto/pulls/comments/9856298"},"html":{"href":"https://github.com/facebook/presto/pull/1044#discussion_r9856298"},"pull_request":{"href":"https://api.github.com/repos/facebook/presto/pulls/1044"}}}}