You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by yu...@apache.org on 2022/07/15 01:31:39 UTC
[pulsar] branch master updated: Correct log in NamespacesBase#internalClearProperties (#16604)
This is an automated email from the ASF dual-hosted git repository.
yuruguo 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 50fc8edc13d Correct log in NamespacesBase#internalClearProperties (#16604)
50fc8edc13d is described below
commit 50fc8edc13d9f698f21a18487d7d1870fc6bb41f
Author: Ruguo Yu <ji...@163.com>
AuthorDate: Fri Jul 15 09:31:30 2022 +0800
Correct log in NamespacesBase#internalClearProperties (#16604)
---
.../main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java | 4 ++--
.../src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java
index 1766874322c..cdfd6a735cf 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java
@@ -2784,11 +2784,11 @@ public abstract class NamespacesBase extends AdminResource {
return policies;
}).thenAccept(v -> {
asyncResponse.resume(Response.noContent().build());
- log.info("[{}] Successfully clear {} properties for on namespace {}", clientAppId(), clearedCount.get(),
+ log.info("[{}] Successfully clear {} properties on namespace {}", clientAppId(), clearedCount.get(),
namespaceName);
}).exceptionally(ex -> {
Throwable cause = ex.getCause();
- log.error("[{}] Failed to remove property for key {} on namespace {}", clientAppId(), clearedCount.get(),
+ log.error("[{}] Failed to clear {} properties on namespace {}", clientAppId(), clearedCount.get(),
namespaceName, cause);
asyncResponse.resume(cause);
return null;
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
index 6036b0a6dec..aa3698e1c7d 100644
--- a/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
+++ b/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/Namespaces.java
@@ -2283,7 +2283,7 @@ public class Namespaces extends NamespacesBase {
@DELETE
@Path("/{tenant}/{namespace}/properties")
- @ApiOperation(value = "Get property value for a given key on a namespace.")
+ @ApiOperation(value = "Clear properties on a given namespace.")
@ApiResponses(value = { @ApiResponse(code = 403, message = "Don't have admin permission"),
@ApiResponse(code = 404, message = "Tenant or namespace doesn't exist"), })
public void clearProperties(