clebertsuconic commented on a change in pull request #3833:
URL: https://github.com/apache/activemq-artemis/pull/3833#discussion_r741393339



##########
File path: 
artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/management/ActiveMQServerControl.java
##########
@@ -1090,7 +1090,7 @@ void destroyQueue(@Parameter(name = "name", desc = "Name 
of the queue to destroy
    @Operation(desc = "Destroy a queue", impact = MBeanOperationInfo.ACTION)
    void destroyQueue(@Parameter(name = "name", desc = "Name of the queue to 
destroy") String name,
                      @Parameter(name = "removeConsumers", desc = "Remove 
consumers of this queue") boolean removeConsumers,
-                     @Parameter(name = "autoDeleteAddress", desc = 
"Automatically delete the address if this was the last queue") boolean 
autoDeleteAddress) throws Exception;
+                     @Parameter(name = "forceAutoDeleteAddress", desc = 
"Automatically delete the address if this was the last queue") boolean 
autoDeleteAddress) throws Exception;

Review comment:
       @gemmellr we could call it instantAutoDeleteAddress instead of force.
   
   This is just doing the check on the address instantly as opposed to leave it 
to the reaper (in the new semantics now).




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to