[#1823] Align DestinationView and DestinationViewMBean (#1824) (#1841)

(cherry picked from commit 5658dc7)

Author: mattrpav

activemq-broker/src/main/java/org/apache/activemq/broker/jmx/DestinationView.java

@@ -137,6 +137,12 @@ public long getStoreMessageSize() {
         return messageStore != null ? messageStore.getMessageStoreStatistics().getMessageSize().getTotalSize() : 0;
     }
 
+    @Override
+    public long getMessagesCachedCount() {
+        return destination.getDestinationStatistics().getMessagesCached().getCount();
+    }
+
+    @Deprecated(forRemoval = true) // see: getMessagesCachedCount() instead. This method name is inconsistent
     public long getMessagesCached() {
         return destination.getDestinationStatistics().getMessagesCached().getCount();
     }
@@ -438,10 +444,12 @@ public int getMaxProducersToAudit() {
         return destination.getMaxProducersToAudit();
     }
 
+    @Override
     public boolean isEnableAudit() {
         return destination.isEnableAudit();
     }
 
+    @Override
     public void setEnableAudit(boolean enableAudit) {
         destination.setEnableAudit(enableAudit);
     }

activemq-broker/src/main/java/org/apache/activemq/broker/jmx/DestinationViewMBean.java

@@ -131,6 +131,13 @@ public interface DestinationViewMBean {
     @MBeanInfo("Number of producers attached to this destination")
     long getProducerCount();
 
+
+    /**
+     * @return the number of messages cached
+     */
+    @MBeanInfo("Number of messages cached")
+    public long getMessagesCachedCount();
+
     /**
      * Returns the number of messages in this destination which are yet to be
      * consumed
@@ -263,6 +270,19 @@ public String sendTextMessageWithProperties(@MBeanInfo("properties") String prop
     @MBeanInfo("Memory used by undelivered messages in bytes")
     long getMemoryUsageByteCount();
 
+    /**
+     * @return the EnableAudit setting for duplicate message checking
+     */
+    @MBeanInfo("Audit is enabled for duplicate message checking")
+    boolean isEnableAudit();
+
+    /**
+     * set the flag to enable or disable audit for duplicate message checking
+     * @param enableAudit
+     * enable/disable audit on the destination
+     */
+    void setEnableAudit(boolean enableAudit);
+
     /**
      * @return the amount of memory allocated to this destination
      */