Skip to content

[Doc] add explanations for setting bookie rack name #13913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Anonymitaet merged 5 commits into from
Jan 25, 2022
Merged

[Doc] add explanations for setting bookie rack name #13913

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3453161
[Doc] add explanations for setting bookie rack name
Anonymitaet Jan 24, 2022
58fd3e6
update
Anonymitaet Jan 24, 2022
50339e3
update
Anonymitaet Jan 24, 2022
f1935fb
update
Anonymitaet Jan 24, 2022
a8e4124
update
Anonymitaet Jan 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdBookies.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,13 @@ private class UpdateBookie extends CliCommand {
description = "Bookie address (format: `address:port`)", required = true)
private String bookieAddress;

@Parameter(names = { "-r", "--rack" }, description = "Bookie rack name", required = true)
@Parameter(names = { "-r", "--rack" }, description = "Bookie rack name. "
+ "If you set a bookie rack name to slash (/) "
+ "or an empty string (\"\"): "
+ "if you use Pulsar earlier than 2.7.5, 2.8.3, and 2.9.2, "
+ "an an exception is thrown; "
+ "if you use Pulsar later than 2.7.5, 2.8.3, and 2.9.2, "
+ "it falls back to /default-rack or /default-region/default-rack.", required = true)
private String bookieRack;

@Parameter(names = { "--hostname" }, description = "Bookie host name", required = false)
Expand Down
4 changes: 4 additions & 0 deletions site2/docs/administration-isolation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,10 @@ bin/pulsar-admin namespaces set-bookie-affinity-group public/default \
--primary-group group-bookie1
```

> **Note**
>
> Do not set a bookie rack name to slash (`/`) or an empty string (`""`) if you use Pulsar earlier than 2.7.5, 2.8.3, and 2.9.2. For the bookie rack name restrictions, see [pulsar-admin bookies set-bookie-rack](https://pulsar.apache.org/tools/pulsar-admin/).

<!--REST API-->

[POST /admin/v2/namespaces/{tenant}/{namespace}/persistence/bookieAffinity](https://pulsar.apache.org/admin-rest-api/?version=master&apiversion=v2#operation/setBookieAffinityGroup)
Expand Down