You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2025/11/07 05:15:41 UTC
[kudu-CR] [docs] refresh `kudu master add` documentation
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23649
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused people who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading people who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
---
M docs/administration.adoc
1 file changed, 34 insertions(+), 16 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/23649/1
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 2:
Build Started http://jenkins.kudu.apache.org/job/pre_commit/2112/
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:00:38 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Abhishek Chennaka (Code Review)" <ge...@cloudera.org>.
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@642
PS1, Line 642:
: port::
> I'm not even sure it's possible to update it directly in the database that backs the HMS without additional knowledge on the format of the data that Impala stores in the HMS for Kudu tables.
It is the same format as with HMS integration enabled. We used to do that in older versions of Kudu without HMS integration. Once we run invalidate metadata in Impala shell (documented below) Impala's set of masters would be updated with the new set of masters.
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 18:08:19 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3: Verified+1
Build Successful
http://jenkins.kudu.apache.org/job/pre_commit/2113/ : SUCCESS
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:51:55 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Marton Greber, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23649
to look at the new patch set (#5).
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused users who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading users who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
---
M docs/administration.adoc
1 file changed, 41 insertions(+), 18 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/23649/5
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 5:
Build Started http://jenkins.kudu.apache.org/job/pre_commit/2118/
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:30:01 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused users who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading users who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Reviewed-on: http://gerrit.cloudera.org:8080/23649
Tested-by: Kudu Jenkins
Reviewed-by: Abhishek Chennaka <ac...@cloudera.com>
---
M docs/administration.adoc
1 file changed, 41 insertions(+), 18 deletions(-)
Approvals:
Kudu Jenkins: Verified
Abhishek Chennaka: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 2: Verified+1
Build Successful
http://jenkins.kudu.apache.org/job/pre_commit/2112/ : SUCCESS
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:01:00 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23649
to look at the new patch set (#3).
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused users who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading users who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
---
M docs/administration.adoc
1 file changed, 43 insertions(+), 21 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/23649/3
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Marton Greber (Code Review)" <ge...@cloudera.org>.
Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 13:35:33 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/23649/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/23649/1//COMMIT_MSG@21
PS1, Line 21: people
> nit: users
Done
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@550
PS1, Line 550: or five node master
> Move to next line
Done
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@581
PS1, Line 581: Define and record all the rest of the customized flags the new master should
: run with once added into the cluster.
> Should we also add a one liner or modify this to advise users to record the
I guess it's just one of the ways to create the list of the flags for a new master: there might be different approaches. I think we better leave this generic part out of the picture, focusing on specifics: the section isn't focusing on best practices per se, but rather on essential steps to successfully add a new Kudu master using the `kudu master add` tool.
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@598
PS1, Line 598: file.
> Move to next line
Done
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@601
PS1, Line 601: is shutdown
> is still in a shutdown state?
I guess it's an attempt to explain that the procedure runs a new master (i.e. kudu-master process), and then it shuts the new master down regardless of the success/failure outcome.
I rephrased it a bit, hopefully it's clear now.
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@610
PS1, Line 610: run
> run with ?
Done
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:00:40 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Abhishek Chennaka (Code Review)" <ge...@cloudera.org>.
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@608
PS1, Line 608: will run with after the procedure is complete:
needed to run the new Kudu master after the procedure is complete.
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@642
PS1, Line 642:
: port::
As per my understanding, with or without HMS integration, we will have kudu.master_addresses populated for every table in HMS database. Only difference is who populates it:
With HMS integration - Kudu talks to HMS database directly.
-> In this case we need to update the information in the HMS database manually.
Without HMS integration - Impala update Kudu entries on Kudu behalf.
-> In this case we can either update the HMS database OR have Impala do it by running the ALTER TABLE query for each table.
I could be missing something as I haven't verified this behavior.
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:28:15 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 4:
Build Started http://jenkins.kudu.apache.org/job/pre_commit/2117/
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 18:14:00 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Abhishek Chennaka (Code Review)" <ge...@cloudera.org>.
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/23649/4/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/4/docs/administration.adoc@650
PS4, Line 650: otherwise update the master addresses for all Kudu tables by running
: corresponding DDL statements in `impala-shell`
My suggestion is to not change anything after line 645.
Reason: With this change, if a user is having HMS integration, they would need to run the ALTER TABLE in impala and that is the only way. But they could also update the masters in HMS database for all the tables with one query which is what I have seen users perform the most.
Let me know what you think, thanks!
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:15:59 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1:
Build Started http://jenkins.kudu.apache.org/job/pre_commit/2105/
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 05:15:50 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Ashwani Raina (Code Review)" <ge...@cloudera.org>.
Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1:
(6 comments)
Looks good to me!
Just a few nits.
http://gerrit.cloudera.org:8080/#/c/23649/1//COMMIT_MSG
Commit Message:
http://gerrit.cloudera.org:8080/#/c/23649/1//COMMIT_MSG@21
PS1, Line 21: people
nit: users
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@550
PS1, Line 550: or five node master
Move to next line
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@581
PS1, Line 581: Define and record all the rest of the customized flags the new master should
: run with once added into the cluster.
Should we also add a one liner or modify this to advise users to record the flags from existing and running kudu-master's flags and customise them for new master?
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@598
PS1, Line 598: file.
Move to next line
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@601
PS1, Line 601: is shutdown
is still in a shutdown state?
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@610
PS1, Line 610: run
run with ?
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 11:14:10 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23649
to look at the new patch set (#2).
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused users who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading users who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
---
M docs/administration.adoc
1 file changed, 43 insertions(+), 21 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/23649/2
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 5: Verified+1
Build Successful
http://jenkins.kudu.apache.org/job/pre_commit/2118/ : SUCCESS
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:30:22 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3:
Build Started http://jenkins.kudu.apache.org/job/pre_commit/2113/
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:51:33 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Ashwani Raina (Code Review)" <ge...@cloudera.org>.
Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
LGTM! Just a small nit.
http://gerrit.cloudera.org:8080/#/c/23649/3/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/3/docs/administration.adoc@605
PS3, Line 605: successful
successful or not
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 10 Nov 2025 13:15:56 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@581
PS1, Line 581: by `--fs_wal_dir` and `--fs_data_dirs`.
: * Define and record all the rest of the
> I guess it's just one of the ways to create the list of the flags for a new
Actually, after some consideration, I added the suggested one-liner into PS4. The reasoning is: it shouldn't hurt even if it's a bit generic.
I hope it looks better now.
http://gerrit.cloudera.org:8080/#/c/23649/3/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/3/docs/administration.adoc@605
PS3, Line 605: successful
> successful or not
Done
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 18:15:27 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@642
PS1, Line 642: if HMS integration is
: enabled
> > I'm not even sure it's possible to update it directly in the database tha
Right: I'd assume that's the same format as with HMS integration enabled if using relatively recent Impala version. I thought this procedure is written having in mind that Impala might be of old/prior versions where it used a different notation for its metadata stored in the HMS.
As for this documentation blurb, what is your suggestion/recommendation as of PS4? Please let me know if you think we need to update it in some particular way. Thank you!
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 18:23:22 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 4: Verified+1
Build Successful
http://jenkins.kudu.apache.org/job/pre_commit/2117/ : SUCCESS
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 18:14:21 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@608
PS1, Line 608: the new Kudu master will be running after the procedure is complete
> needed to run the new Kudu master after the procedure is complete.
Done
http://gerrit.cloudera.org:8080/#/c/23649/1/docs/administration.adoc@642
PS1, Line 642: if HMS integration is
: enabled
> As per my understanding, with or without HMS integration, we will have kudu
Yes, that's indeed so, AFAIK. However, I don't see now this contradicts the described procedure or would require modifying it in any way. I wouldn't recommend messing up with the metadata that Impala manages on is own in the HMS in the latter case. I'm not even sure it's possible to update it directly in the database that backs the HMS without additional knowledge on the format of the data that Impala stores in the HMS for Kudu tables.
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 23:51:50 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Kudu Jenkins (Code Review)" <ge...@cloudera.org>.
Kudu Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 1: Verified+1
Build Successful
http://jenkins.kudu.apache.org/job/pre_commit/2105/ : SUCCESS
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 07 Nov 2025 05:16:11 +0000
Gerrit-HasComments: No
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Marton Greber, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23649
to look at the new patch set (#4).
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
[docs] refresh `kudu master add` documentation
This changelist updates the documentation on using the `kudu master add`
CLI tool (which is effectively obsolete starting with Kudu 1.16.0).
It turned out the original version was quite misleading. There is an
evidence that it confused users who attempted it because of the
omission of a few crucial points, especially if running the procedure
in a secure (i.e. Kerberized) Kudu cluster.
This changelist addresses the issue, at least clarifying on the
essential points of the procedure. At the time of writing, the current
Kudu version is 1.18.0, and the `kudu master add` CLI tool is obsolete.
However, recently I found that there are still Kudu clusters of the
1.15.0 version in the wild, so I thought it makes sense to update the
docs on the procedure to avoid confusing and misleading users who might
try applying it at their 1.15.0 Kudu clusters.
This is a follow-up to 336c65bfd84a5115dc5c6be6521751d53aa0c286.
Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
---
M docs/administration.adoc
1 file changed, 46 insertions(+), 21 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/23649/4
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/23649/4/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/23649/4/docs/administration.adoc@650
PS4, Line 650: otherwise update the master addresses for all Kudu tables by running
: corresponding DDL statements in `impala-shell`
> My suggestion is to not change anything after line 645.
So, we aren't targeting any cases when people use newer Kudu and older Impala/HMS then. That's totally fine I and guess that's quite unlikely that many people have such a configuration in any case.
Done.
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:30:34 +0000
Gerrit-HasComments: Yes
[kudu-CR] [docs] refresh `kudu master add` documentation
Posted by "Abhishek Chennaka (Code Review)" <ge...@cloudera.org>.
Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23649 )
Change subject: [docs] refresh `kudu master add` documentation
......................................................................
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/23649
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d2ae84d596a0c3ffe840cc16d3ac0d0d28d05bc
Gerrit-Change-Number: 23649
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:31:46 +0000
Gerrit-HasComments: No