Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22527 )
Change subject: IMPALA-13820: add ipv6 support for webui/hs2/hs2-http/beeswax ...................................................................... Patch Set 24: Code-Review+1 (4 comments) http://gerrit.cloudera.org:8080/#/c/22527/23/shell/impala_shell/impala_client.py File shell/impala_shell/impala_client.py: http://gerrit.cloudera.org:8080/#/c/22527/23/shell/impala_shell/impala_client.py@432 PS23, Line 432: host_and_port = self._to_host_port(self.impalad_host, self.impalad_port) : assert self.http_path > Done Done http://gerrit.cloudera.org:8080/#/c/22527/23/tests/common/impala_service.py File tests/common/impala_service.py: http://gerrit.cloudera.org:8080/#/c/22527/23/tests/common/impala_service.py@73 PS23, Line 73: host_port = to_host_port(self.webserver_interface, self.webserver_port) : url = "%s://%s/%s" % (protocol, host_port, page_name) > Done Done http://gerrit.cloudera.org:8080/#/c/22527/23/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: http://gerrit.cloudera.org:8080/#/c/22527/23/tests/common/impala_test_suite.py@505 PS23, Line 505: @classmethod > The problem is that this function and create_impala_client() are called in Sorry, I should highlight L425, but that still answer my question. http://gerrit.cloudera.org:8080/#/c/22527/23/tests/metadata/test_event_processing_base.py File tests/metadata/test_event_processing_base.py: http://gerrit.cloudera.org:8080/#/c/22527/23/tests/metadata/test_event_processing_base.py@33 PS23, Line 33: @classmethod : def _run_test_insert_events_impl(cls, suite, unique_database, is_transactional=False): > I agree that these shouldn't be class methods. I think that the original re Ack -- To view, visit http://gerrit.cloudera.org:8080/22527 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I51ac66c568cc9bb06f4a3915db07a53c100109b6 Gerrit-Change-Number: 22527 Gerrit-PatchSet: 24 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: gaurav singh <[email protected]> Gerrit-Comment-Date: Wed, 28 May 2025 17:44:30 +0000 Gerrit-HasComments: Yes
