anujphadke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9619 )
Change subject: IMPALA-5695: restart_first_impalad() does not start impalad ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9619/1/tests/authorization/test_grant_revoke.py File tests/authorization/test_grant_revoke.py: http://gerrit.cloudera.org:8080/#/c/9619/1/tests/authorization/test_grant_revoke.py@80 PS1, Line 80: IMPALA_HOME = os.environ['IMPALA_HOME'] : IMPALAD_PATH = os.path.join(IMPALA_HOME, : 'bin/start-impalad.sh --server_name=server1') > The issue is that impalad.restart() remains broken. I think the proper fix Thanks Adam and Freddy! I agree. I looked into it and the issue does not happen if you just set shell=True in the popen in exec_process_async. I am still digging up the code to root cause why this works. I will post my updates soon. -- To view, visit http://gerrit.cloudera.org:8080/9619 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I30a5c25e132deebb9731891364196af31e57d875 Gerrit-Change-Number: 9619 Gerrit-PatchSet: 1 Gerrit-Owner: anujphadke <apha...@cloudera.com> Gerrit-Reviewer: Adam Holley <g...@holleyism.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: anujphadke <apha...@cloudera.com> Gerrit-Comment-Date: Fri, 16 Mar 2018 02:24:18 +0000 Gerrit-HasComments: Yes