{"action":"created","issue":{"url":"https://api.github.com/repos/graalvm/truffle/issues/98","repository_url":"https://api.github.com/repos/graalvm/truffle","labels_url":"https://api.github.com/repos/graalvm/truffle/issues/98/labels{/name}","comments_url":"https://api.github.com/repos/graalvm/truffle/issues/98/comments","events_url":"https://api.github.com/repos/graalvm/truffle/issues/98/events","html_url":"https://github.com/graalvm/truffle/pull/98","id":138623325,"number":98,"title":"KillException and QuitException handling","user":{"login":"mlvdv","id":12618713,"avatar_url":"https://avatars.githubusercontent.com/u/12618713?v=3","gravatar_id":"","url":"https://api.github.com/users/mlvdv","html_url":"https://github.com/mlvdv","followers_url":"https://api.github.com/users/mlvdv/followers","following_url":"https://api.github.com/users/mlvdv/following{/other_user}","gists_url":"https://api.github.com/users/mlvdv/gists{/gist_id}","starred_url":"https://api.github.com/users/mlvdv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlvdv/subscriptions","organizations_url":"https://api.github.com/users/mlvdv/orgs","repos_url":"https://api.github.com/users/mlvdv/repos","events_url":"https://api.github.com/users/mlvdv/events{/privacy}","received_events_url":"https://api.github.com/users/mlvdv/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/graalvm/truffle/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/graalvm/truffle/labels/help%20wanted","name":"help wanted","color":"159818"},{"url":"https://api.github.com/repos/graalvm/truffle/labels/oracle-emp","name":"oracle-emp","color":"bfe5bf"}],"state":"open","locked":false,"assignee":{"login":"chumer","id":4346215,"avatar_url":"https://avatars.githubusercontent.com/u/4346215?v=3","gravatar_id":"","url":"https://api.github.com/users/chumer","html_url":"https://github.com/chumer","followers_url":"https://api.github.com/users/chumer/followers","following_url":"https://api.github.com/users/chumer/following{/other_user}","gists_url":"https://api.github.com/users/chumer/gists{/gist_id}","starred_url":"https://api.github.com/users/chumer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chumer/subscriptions","organizations_url":"https://api.github.com/users/chumer/orgs","repos_url":"https://api.github.com/users/chumer/repos","events_url":"https://api.github.com/users/chumer/events{/privacy}","received_events_url":"https://api.github.com/users/chumer/received_events","type":"User","site_admin":false},"milestone":null,"comments":9,"created_at":"2016-03-05T00:14:55Z","updated_at":"2016-03-10T11:06:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/graalvm/truffle/pulls/98","html_url":"https://github.com/graalvm/truffle/pull/98","diff_url":"https://github.com/graalvm/truffle/pull/98.diff","patch_url":"https://github.com/graalvm/truffle/pull/98.patch"},"body":"Clarify and document required behavior for WrapperNodes in the instrumentation framework. Fix a related bug in ProbeNode.  Add tests for correct handling.\r\n@chumer please have a look at tests failing for automatically generated wrappers"},"comment":{"url":"https://api.github.com/repos/graalvm/truffle/issues/comments/194793047","html_url":"https://github.com/graalvm/truffle/pull/98#issuecomment-194793047","issue_url":"https://api.github.com/repos/graalvm/truffle/issues/98","id":194793047,"user":{"login":"woess","id":6517543,"avatar_url":"https://avatars.githubusercontent.com/u/6517543?v=3","gravatar_id":"","url":"https://api.github.com/users/woess","html_url":"https://github.com/woess","followers_url":"https://api.github.com/users/woess/followers","following_url":"https://api.github.com/users/woess/following{/other_user}","gists_url":"https://api.github.com/users/woess/gists{/gist_id}","starred_url":"https://api.github.com/users/woess/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/woess/subscriptions","organizations_url":"https://api.github.com/users/woess/orgs","repos_url":"https://api.github.com/users/woess/repos","events_url":"https://api.github.com/users/woess/events{/privacy}","received_events_url":"https://api.github.com/users/woess/received_events","type":"User","site_admin":false},"created_at":"2016-03-10T11:06:04Z","updated_at":"2016-03-10T11:06:04Z","body":"@smarr: this exception should be handled in the framework (inside ProbeNode) and not propagated to the actual instrumentation observers' onReturnExceptional methods. This way we don't duplicate the catch/rethrow (there are multiple wrapper classes (at least one per language) but only 1 ProbeNode) and effectively make it an implementation detail *here*."}}