Michael Ho has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11113 )

Change subject: IMPALA-7390: Pin rpc-mgr-kerberized-test to localhost.
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11113/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11113/1//COMMIT_MSG@11
PS1, Line 11: failing due to DNS resolution reasons
Does it mean in general, to run Impala with docker, one has to specify the 
hostname ? Is there anything we can do at this line 
(https://github.com/apache/impala/blob/master/be/src/common/init.cc#L191) to 
fix the issue ?


http://gerrit.cloudera.org:8080/#/c/11113/1/be/src/rpc/rpc-mgr-kerberized-test.cc
File be/src/rpc/rpc-mgr-kerberized-test.cc:

http://gerrit.cloudera.org:8080/#/c/11113/1/be/src/rpc/rpc-mgr-kerberized-test.cc@86
PS1, Line 86: FLAGS_hostname = "localhost";
nit: may help to add a comment on why InitCommonRuntime() doesn't set this up 
correctly.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I91003cbc86177feb7f99563f61297f7da7fabab4
Gerrit-Change-Number: 11113
Gerrit-PatchSet: 1
Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Comment-Date: Fri, 03 Aug 2018 16:48:11 +0000
Gerrit-HasComments: Yes

Reply via email to