Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17849 )
Change subject: [util] allow DnsResolver to refresh DNS addresses ...................................................................... Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/17849/1/src/kudu/util/net/net_util.cc File src/kudu/util/net/net_util.cc: http://gerrit.cloudera.org:8080/#/c/17849/1/src/kudu/util/net/net_util.cc@211 PS1, Line 211: port_ Okay so we are summing the override flag doesn't pass port? -- To view, visit http://gerrit.cloudera.org:8080/17849 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0616f3e6fb50aba271f106b05d1926fc46a53ed0 Gerrit-Change-Number: 17849 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 17 Sep 2021 22:21:43 +0000 Gerrit-HasComments: Yes
