{"action":"created","issue":{"url":"https://api.github.com/repos/grpc/grpc-java/issues/836","labels_url":"https://api.github.com/repos/grpc/grpc-java/issues/836/labels{/name}","comments_url":"https://api.github.com/repos/grpc/grpc-java/issues/836/comments","events_url":"https://api.github.com/repos/grpc/grpc-java/issues/836/events","html_url":"https://github.com/grpc/grpc-java/pull/836","id":101926226,"number":836,"title":"Remove proto size restriction when parsing protos.","user":{"login":"nmittler","id":6493296,"avatar_url":"https://avatars.githubusercontent.com/u/6493296?v=3","gravatar_id":"","url":"https://api.github.com/users/nmittler","html_url":"https://github.com/nmittler","followers_url":"https://api.github.com/users/nmittler/followers","following_url":"https://api.github.com/users/nmittler/following{/other_user}","gists_url":"https://api.github.com/users/nmittler/gists{/gist_id}","starred_url":"https://api.github.com/users/nmittler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nmittler/subscriptions","organizations_url":"https://api.github.com/users/nmittler/orgs","repos_url":"https://api.github.com/users/nmittler/repos","events_url":"https://api.github.com/users/nmittler/events{/privacy}","received_events_url":"https://api.github.com/users/nmittler/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-19T15:56:35Z","updated_at":"2015-08-20T03:29:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/grpc/grpc-java/pulls/836","html_url":"https://github.com/grpc/grpc-java/pull/836","diff_url":"https://github.com/grpc/grpc-java/pull/836.diff","patch_url":"https://github.com/grpc/grpc-java/pull/836.patch"},"body":"@ejona86 PTAL .. this is the first part of the change to support maxMessageSize."},"comment":{"url":"https://api.github.com/repos/grpc/grpc-java/issues/comments/132874413","html_url":"https://github.com/grpc/grpc-java/pull/836#issuecomment-132874413","issue_url":"https://api.github.com/repos/grpc/grpc-java/issues/836","id":132874413,"user":{"login":"ejona86","id":2811396,"avatar_url":"https://avatars.githubusercontent.com/u/2811396?v=3","gravatar_id":"","url":"https://api.github.com/users/ejona86","html_url":"https://github.com/ejona86","followers_url":"https://api.github.com/users/ejona86/followers","following_url":"https://api.github.com/users/ejona86/following{/other_user}","gists_url":"https://api.github.com/users/ejona86/gists{/gist_id}","starred_url":"https://api.github.com/users/ejona86/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejona86/subscriptions","organizations_url":"https://api.github.com/users/ejona86/orgs","repos_url":"https://api.github.com/users/ejona86/repos","events_url":"https://api.github.com/users/ejona86/events{/privacy}","received_events_url":"https://api.github.com/users/ejona86/received_events","type":"User","site_admin":false},"created_at":"2015-08-20T03:29:24Z","updated_at":"2015-08-20T03:29:24Z","body":"Do we want to do the same thing that proto does if the message was incomplete?\r\n\r\n```java\r\n    try {\r\n      codedInput.checkLastTagWas(0);\r\n    } catch (InvalidProtocolBufferException e) {\r\n      throw e.setUnfinishedMessage(message);\r\n    }\r\n```\r\n\r\nThat does actually look useful, as you can piece together what the message was a bit, but aren't accidentally going to trust it. That does seem useful for debugging."}}