Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16157 )

Change subject: IMPALA-5534: Fix and enable experimental failure tests
......................................................................


Patch Set 2: Code-Review+2

(2 comments)

http://gerrit.cloudera.org:8080/#/c/16157/2/tests/custom_cluster/test_catalog_hms_failures.py
File tests/custom_cluster/test_catalog_hms_failures.py:

http://gerrit.cloudera.org:8080/#/c/16157/2/tests/custom_cluster/test_catalog_hms_failures.py@17
PS2, Line 17: # Tests to validate the Catalog Service continues to function 
even if the HMS fails.
nit, May be move this line to the python way of documentation on line 36 as 
well?


http://gerrit.cloudera.org:8080/#/c/16157/2/tests/custom_cluster/test_process_failures.py
File tests/custom_cluster/test_process_failures.py:

http://gerrit.cloudera.org:8080/#/c/16157/2/tests/custom_cluster/test_process_failures.py@143
PS2, Line 143: IMPALA-9848
Thanks for writing a test case and creating a JIRA for this. I will try to look 
into whats happening there.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9dbb98017fb6c40cea349e7c63a35c325cbbc288
Gerrit-Change-Number: 16157
Gerrit-PatchSet: 2
Gerrit-Owner: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Comment-Date: Fri, 10 Jul 2020 17:48:55 +0000
Gerrit-HasComments: Yes

Reply via email to