Anubhav Jindal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24372 )

Change subject: IMPALA-15043: Add Kubernetes-in-Docker E2E scripts
......................................................................


Patch Set 47:

(2 comments)

Done

http://gerrit.cloudera.org:8080/#/c/24372/45/tests/common/impala_test_suite.py
File tests/common/impala_test_suite.py:

http://gerrit.cloudera.org:8080/#/c/24372/45/tests/common/impala_test_suite.py@495
PS45, Line 495:       except Exception as e:
> This now ignores Beeswax client setup failures unconditionally.
Agreed. I gated Beeswax setup fallback on --testing_remote_cluster


http://gerrit.cloudera.org:8080/#/c/24372/45/tests/common/impala_test_suite.py@524
PS45, Line 524:     default_protocol = cls.default_test_protocol()
> This fallback changes test semantics by silently switching protocols when t
Makes sense, updated. We now fail when the requested default protocol client is 
unavailable in normal runs



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I122b99c38c8c70ab535349f92212621330f2aa55
Gerrit-Change-Number: 24372
Gerrit-PatchSet: 47
Gerrit-Owner: Anubhav Jindal <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Anubhav Jindal <[email protected]>
Gerrit-Reviewer: Gokul Kolady <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Comment-Date: Tue, 28 Jul 2026 21:38:51 +0000
Gerrit-HasComments: Yes

Reply via email to