You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by te...@apache.org on 2024/03/27 16:08:00 UTC
(pulsar) branch master updated: [improve][broker] Optimize web interface deleteDynamicConfiguration return error message (#22356)
This is an automated email from the ASF dual-hosted git repository.
technoboy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new e4553391f96 [improve][broker] Optimize web interface deleteDynamicConfiguration return error message (#22356)
e4553391f96 is described below
commit e4553391f96af3bda3d8252b97cac3de1f39a1b5
Author: hanmz <gu...@tencent.com>
AuthorDate: Thu Mar 28 00:07:54 2024 +0800
[improve][broker] Optimize web interface deleteDynamicConfiguration return error message (#22356)
---
.../src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java
index 61b354610ac..83067e9f296 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java
@@ -526,7 +526,7 @@ public class BrokersBase extends AdminResource {
private CompletableFuture<Void> internalDeleteDynamicConfigurationOnMetadataAsync(String configName) {
if (!pulsar().getBrokerService().isDynamicConfiguration(configName)) {
- throw new RestException(Status.PRECONDITION_FAILED, " Can't update non-dynamic configuration");
+ throw new RestException(Status.PRECONDITION_FAILED, "Can't delete non-dynamic configuration");
} else {
return dynamicConfigurationResources().setDynamicConfigurationAsync(old -> {
if (old != null) {