Wang Xixu has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18567 )

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


Patch Set 8:

(6 comments)

> Patch Set 7:
>
> (6 comments)

http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@9
PS7, Line 9: n server startup
> nit: at server startup
Done


http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@9
PS7, Line 9: config fi
> nit: config
Done


http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@10
PS7, Line 10: config,
> nit: config
Done


http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@11
PS7, Line 11: Therefore it will
            : be good to keep case insensitive for DNS name.
> Maybe you mean DNS names are  case insensitive?
Yes


http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@15
PS7, Line 15: when rewriting Raft configure, user will input DNS name, which
> As far as I know, when using 'kudu local_replica cmeta rewrite_raft_config'
Yes.


http://gerrit.cloudera.org:8080/#/c/18567/7//COMMIT_MSG@15
PS7, Line 15: configure
> nit: config
Done



--
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: comment
Gerrit-Change-Id: If955e91a9e081dad7aebdcc497c413c8eb10eb4b
Gerrit-Change-Number: 18567
Gerrit-PatchSet: 8
Gerrit-Owner: Wang Xixu <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Comment-Date: Mon, 20 Jun 2022 07:11:15 +0000
Gerrit-HasComments: Yes

Reply via email to