{"comment":{"url":"https://api.github.com/repos/sudheeshkatkam/drill/comments/12280019","html_url":"https://github.com/sudheeshkatkam/drill/commit/6ec7b326459094a865846b1758e9d4457752af4c#commitcomment-12280019","id":12280019,"user":{"login":"jaltekruse","id":365366,"avatar_url":"https://avatars.githubusercontent.com/u/365366?v=3","gravatar_id":"","url":"https://api.github.com/users/jaltekruse","html_url":"https://github.com/jaltekruse","followers_url":"https://api.github.com/users/jaltekruse/followers","following_url":"https://api.github.com/users/jaltekruse/following{/other_user}","gists_url":"https://api.github.com/users/jaltekruse/gists{/gist_id}","starred_url":"https://api.github.com/users/jaltekruse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jaltekruse/subscriptions","organizations_url":"https://api.github.com/users/jaltekruse/orgs","repos_url":"https://api.github.com/users/jaltekruse/repos","events_url":"https://api.github.com/users/jaltekruse/events{/privacy}","received_events_url":"https://api.github.com/users/jaltekruse/received_events","type":"User","site_admin":false},"position":23,"line":44,"path":"exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptions.java","commit_id":"6ec7b326459094a865846b1758e9d4457752af4c","created_at":"2015-07-21T16:55:57Z","updated_at":"2015-07-21T16:55:57Z","body":"I'm inclined to encourage checking messages as well as exception types, as we should be trying to move towards a state where Drill throws nothing but user exceptions. We should start to make sure we are executing the expected exception code path, not just an exception path."}}