Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23611 )

Change subject: [test] Skip test if hostname has no DNS record
......................................................................


Patch Set 4: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23611/4/src/kudu/rpc/rpc-test-base.h
File src/kudu/rpc/rpc-test-base.h:

http://gerrit.cloudera.org:8080/#/c/23611/4/src/kudu/rpc/rpc-test-base.h@100
PS4, Line 100: Hostname has no DNS record
This statement is a bit vague since there might be DNS records for the 
hostname, but not of the type required for GetFQDN() to succeed.

Maybe, change this to "Could not find FQDN of this node" or "GetFQDN() failed 
for this host"?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id377c73a752e18324dbd83aad7e613b3ba99ee6b
Gerrit-Change-Number: 23611
Gerrit-PatchSet: 4
Gerrit-Owner: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Comment-Date: Tue, 04 Nov 2025 19:29:08 +0000
Gerrit-HasComments: Yes

Reply via email to