Michael Smith 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@1044
PS1, Line 1044: +
> Yeah, but I'd rather keep it consistent with the rest of the code.
I suspect we should disable this check as it doesn't match to our existing 
patterns.



--
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-Comment-Date: Wed, 05 Feb 2025 21:42:11 +0000
Gerrit-HasComments: Yes

Reply via email to