{"action":"created","issue":{"url":"https://api.github.com/repos/mapr/drill-test-framework/issues/191","repository_url":"https://api.github.com/repos/mapr/drill-test-framework","labels_url":"https://api.github.com/repos/mapr/drill-test-framework/issues/191/labels{/name}","comments_url":"https://api.github.com/repos/mapr/drill-test-framework/issues/191/comments","events_url":"https://api.github.com/repos/mapr/drill-test-framework/issues/191/events","html_url":"https://github.com/mapr/drill-test-framework/pull/191","id":172686637,"number":191,"title":"Fix a bug which may lead to hang process caused by Utils.execCmd(cmd)","user":{"login":"beyond1920","id":1525333,"avatar_url":"https://avatars.githubusercontent.com/u/1525333?v=3","gravatar_id":"","url":"https://api.github.com/users/beyond1920","html_url":"https://github.com/beyond1920","followers_url":"https://api.github.com/users/beyond1920/followers","following_url":"https://api.github.com/users/beyond1920/following{/other_user}","gists_url":"https://api.github.com/users/beyond1920/gists{/gist_id}","starred_url":"https://api.github.com/users/beyond1920/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/beyond1920/subscriptions","organizations_url":"https://api.github.com/users/beyond1920/orgs","repos_url":"https://api.github.com/users/beyond1920/repos","events_url":"https://api.github.com/users/beyond1920/events{/privacy}","received_events_url":"https://api.github.com/users/beyond1920/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-23T12:31:30Z","updated_at":"2016-08-24T05:35:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mapr/drill-test-framework/pulls/191","html_url":"https://github.com/mapr/drill-test-framework/pull/191","diff_url":"https://github.com/mapr/drill-test-framework/pull/191.diff","patch_url":"https://github.com/mapr/drill-test-framework/pull/191.patch"},"body":"#188 Sorry to close the previous pull request accidentally. So open a new pull request.\r\nUtils.execCmd(cmd) hangs sometimes, because both standard error stream and standard output stream of the Process have to be consumed. Otherwise hang will occasionally occurs like ghost. The JDK's Javadoc documentation provides the answer to this question: Because some native platforms only provide limited buffer size for standard input and output streams, failure to promptly write the input stream or read the output stream of the subprocess may cause the subprocess to block, or even deadlock. More details could be found at http://www.javaworld.com/article/2071275/core-java/when-runtime-exec---won-t.html."},"comment":{"url":"https://api.github.com/repos/mapr/drill-test-framework/issues/comments/241962186","html_url":"https://github.com/mapr/drill-test-framework/pull/191#issuecomment-241962186","issue_url":"https://api.github.com/repos/mapr/drill-test-framework/issues/191","id":241962186,"user":{"login":"cchang738","id":4193676,"avatar_url":"https://avatars.githubusercontent.com/u/4193676?v=3","gravatar_id":"","url":"https://api.github.com/users/cchang738","html_url":"https://github.com/cchang738","followers_url":"https://api.github.com/users/cchang738/followers","following_url":"https://api.github.com/users/cchang738/following{/other_user}","gists_url":"https://api.github.com/users/cchang738/gists{/gist_id}","starred_url":"https://api.github.com/users/cchang738/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cchang738/subscriptions","organizations_url":"https://api.github.com/users/cchang738/orgs","repos_url":"https://api.github.com/users/cchang738/repos","events_url":"https://api.github.com/users/cchang738/events{/privacy}","received_events_url":"https://api.github.com/users/cchang738/received_events","type":"User","site_admin":false},"created_at":"2016-08-24T05:35:01Z","updated_at":"2016-08-24T05:35:01Z","body":"@beyond1920 I reviewed your changes. Looks really good. Can you squash your changes into one commit and create a new PR please? Include the ColumnList change as well. I will do more testing here and merge after. Thanks so much. "}}
{"actor":{"display_login":"cchang738"}}