Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22438 )

Change subject: IMPALA-13724: Add hostnames for Docker host and gateway to 
Impala containers
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22438/1/bin/start-impala-cluster.py
File bin/start-impala-cluster.py:

http://gerrit.cloudera.org:8080/#/c/22438/1/bin/start-impala-cluster.py@1021
PS1, Line 1021:     internal_listen_host = os.environ["INTERNAL_LISTEN_HOST"]
There is an argument parser in L59. Can you add an argument that defaults to 
os.environ map, like --tuple_cache_dir?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I545607c0bb32f8043a0d3f6045710f28a47bab99
Gerrit-Change-Number: 22438
Gerrit-PatchSet: 1
Gerrit-Owner: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Thu, 06 Feb 2025 00:27:18 +0000
Gerrit-HasComments: Yes

Reply via email to