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 <gunn...@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) {

Reply via email to