You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Wang Xixu (Code Review)" <ge...@cloudera.org> on 2022/06/20 07:10:38 UTC

[kudu-CR] [Catalog] Make DNS name of Kudu master case insensitive

Hello Tidy Bot, Yingchun Lai, Yifan Zhang, Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18567

to look at the new patch set (#8).

Change subject: [Catalog] Make DNS name of Kudu master case insensitive
......................................................................

[Catalog] Make DNS name of Kudu master case insensitive

Kudu master gets DNS name from config file when server startup
and writes it into sys table. But after rewriting Raft config,
it will be changed with different character case. Therefore it will
be good to keep case insensitive for DNS name.

In our situtation, the config file uses lower case DNS name,
when rewriting Raft configure, user will input DNS name, which
is in upper case. Then Kudu master get an error when starting.

Change-Id: If955e91a9e081dad7aebdcc497c413c8eb10eb4b
---
M src/kudu/master/sys_catalog.cc
M src/kudu/util/net/net_util.h
2 files changed, 15 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/18567/8
-- 
To view, visit http://gerrit.cloudera.org:8080/18567
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If955e91a9e081dad7aebdcc497c413c8eb10eb4b
Gerrit-Change-Number: 18567
Gerrit-PatchSet: 8
Gerrit-Owner: Wang Xixu <14...@qq.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wang Xixu <14...@qq.com>
Gerrit-Reviewer: Yifan Zhang <ch...@163.com>
Gerrit-Reviewer: Yingchun Lai <ac...@gmail.com>