Skip to content

ZOOKEEPER-4902: Document that read-only mode also enables isro 4lw#2231

Closed
cnauroth wants to merge 1 commit intoapache:masterfrom
cnauroth:ZOOKEEPER-4902
Closed

ZOOKEEPER-4902: Document that read-only mode also enables isro 4lw#2231
cnauroth wants to merge 1 commit intoapache:masterfrom
cnauroth:ZOOKEEPER-4902

Conversation

@cnauroth
Copy link
Contributor

@cnauroth cnauroth commented Mar 6, 2025

No description provided.

@cnauroth
Copy link
Contributor Author

cnauroth commented Mar 6, 2025

Code reference showing where this is done:

if (System.getProperty("readonlymode.enabled", "false").equals("true")) {
whiteListedCommands.add("isro");
}

@cnauroth
Copy link
Contributor Author

@anmolnar , @eolivelli , @tisonkun , are any of you available for a code review of this small documentation change? Thank you.

@cnauroth cnauroth closed this in 2d2820e Mar 12, 2025
cnauroth added a commit that referenced this pull request Mar 12, 2025
Closes #2231

Signed-off-by: Zili Chen <tison@apache.org>
Signed-off-by: Kezhu Wang <kezhuw@apache.org>
(cherry picked from commit 2d2820e)
cnauroth added a commit that referenced this pull request Mar 12, 2025
Closes #2231

Signed-off-by: Zili Chen <tison@apache.org>
Signed-off-by: Kezhu Wang <kezhuw@apache.org>
(cherry picked from commit 2d2820e)
(cherry picked from commit 086b787)
@cnauroth cnauroth deleted the ZOOKEEPER-4902 branch March 12, 2025 21:24
@cnauroth
Copy link
Contributor Author

I have committed this to master, branch-3.9 and branch-3.8. Thank you for the reviews, @tisonkun and @kezhuw .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants