Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12472 )
Change subject: IMPALA-8199: Fix stress test: 'No module named RuntimeProfile.ttypes' ...................................................................... IMPALA-8199: Fix stress test: 'No module named RuntimeProfile.ttypes' A recent commit (IMPALA-6964) broke the stress test because it added an import of a generated thrift value to a python file that is included by the stress test. The stress test is intended to be able to be run without doing a full build of Impala, but in this case the generated thrift isn't available, leading to an import error. The solution is to only import the thrift value in the function where it is used, which is not called by the stress test. Testing: - Ran the stress test manually without doing a full build and confirmed that it works now. Change-Id: I7a3bd26d743ef6603fabf92f904feb4677001da5 Reviewed-on: http://gerrit.cloudera.org:8080/12472 Reviewed-by: Thomas Marshall <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/util/parse_util.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Thomas Marshall: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12472 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7a3bd26d743ef6603fabf92f904feb4677001da5 Gerrit-Change-Number: 12472 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
