Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11484 )

Change subject: IMPALA-1760: Implement shutdown command
......................................................................


Patch Set 2:

(2 comments)

Only very minor changes from the previous patch but I'd appreciate a quick look 
to confirm they make sure.

http://gerrit.cloudera.org:8080/#/c/11484/2/tests/custom_cluster/test_restart_services.py
File tests/custom_cluster/test_restart_services.py:

http://gerrit.cloudera.org:8080/#/c/11484/2/tests/custom_cluster/test_restart_services.py@180
PS2, Line 180: tpch
This was changed from tpch_parquet to fix the failure on S3.


http://gerrit.cloudera.org:8080/#/c/11484/2/tests/custom_cluster/test_restart_services.py@269
PS2, Line 269: default_query_options
this was changed to fix the failure on erasure coding



--
To view, visit http://gerrit.cloudera.org:8080/11484
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8f3679ef442745a60a0ab97c4e9eac437aef9463
Gerrit-Change-Number: 11484
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Mon, 24 Sep 2018 21:25:20 +0000
Gerrit-HasComments: Yes

Reply via email to