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

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


Patch Set 3:

(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: import pytest
> nit, May be move this line to the python way of documentation on line 36 as
Done


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
Sounds good thanks. I think Anurag has been looking into this as well.



--
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: 3
Gerrit-Owner: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Comment-Date: Mon, 13 Jul 2020 18:46:30 +0000
Gerrit-HasComments: Yes

Reply via email to