Jennifer88huang commented on a change in pull request #4367: 
[document][swagger]Update topics rest api document
URL: https://github.com/apache/pulsar/pull/4367#discussion_r291825101
 
 

 ##########
 File path: 
pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java
 ##########
 @@ -358,11 +580,26 @@ public void skipMessages(@PathParam("tenant") String 
tenant, @PathParam("namespa
     @POST
     
@Path("/{tenant}/{namespace}/{topic}/subscription/{subName}/expireMessages/{expireTimeInSeconds}")
     @ApiOperation(value = "Expire messages on a topic subscription.")
-    @ApiResponses(value = { @ApiResponse(code = 403, message = "Don't have 
admin permission"),
-            @ApiResponse(code = 404, message = "Topic or subscription does not 
exist") })
-    public void expireTopicMessages(@PathParam("tenant") String tenant, 
@PathParam("namespace") String namespace,
-            @PathParam("topic") @Encoded String encodedTopic, 
@PathParam("subName") String encodedSubName,
+    @ApiResponses(value = {
+            @ApiResponse(code = 401, message = "Don't have permission to 
administrate resources on this tenant or" +
+                    "subscriber is not authorized to access this operation"),
+            @ApiResponse(code = 403, message = "Don't have admin permission"),
+            @ApiResponse(code = 404, message = "Topic or subscription does not 
exist"),
+            @ApiResponse(code = 405, message = "Expire messages on a 
non-persistent topic is not allowed"),
+            @ApiResponse(code = 500, message = "Internal server error"),
+            @ApiResponse(code = 503, message = "Failed to validate global 
cluster configuration") })
+    public void expireTopicMessages(
+            @ApiParam(value = "Specify the tenant", required = true)
+            @PathParam("tenant") String tenant,
+            @ApiParam(value = "Specify the namespace", required = true)
+            @PathParam("namespace") String namespace,
+            @ApiParam(value = "Specify topic name", required = true)
+            @PathParam("topic") @Encoded String encodedTopic,
+            @ApiParam(value = "Subscription to be expire messages on")
 
 Review comment:
   we can re-write this sentence. Do you mean "Subscription to expiry messages"?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to