michaelpearce-gain commented on a change in pull request #3567:
URL: https://github.com/apache/activemq-artemis/pull/3567#discussion_r633910939



##########
File path: artemis-server/src/main/resources/schema/artemis-configuration.xsd
##########
@@ -3638,12 +3638,29 @@
             <xsd:element name="slow-consumer-threshold" type="xsd:long" 
maxOccurs="1" minOccurs="0">
                <xsd:annotation>
                   <xsd:documentation>
-                     The minimum rate of message consumption allowed before a 
consumer is considered "slow." Measured
-                     in messages-per-second.
+                     The minimum rate of message consumption allowed before a 
consumer is considered "slow."  Measurement
+                     unit is defined by the 
slow-consumer-threshold-measurement-unit parameter.  By default this is
+                     messages-per-seconds
                   </xsd:documentation>
                </xsd:annotation>
             </xsd:element>
 
+            <xsd:element name="slow-consumer-threshold-measurement-unit" 
maxOccurs="1" minOccurs="0">
+               <xsd:annotation>
+                  <xsd:documentation>
+                     The units used to measure the slow consumer threshold.  
Default is messages-per-second.
+                  </xsd:documentation>
+               </xsd:annotation>
+               <xsd:simpleType>
+                  <xsd:restriction base="xsd:string">

Review comment:
       e.g. could just have a shared type in the xsd that defines timeUnit and 
simple share that between Cleberts journal history time unit / period and yours.
   
       <xsd:simpleType name="timeUnit">
           <xsd:restriction base="xsd:string">
               <xsd:enumeration value="DAYS"/>
               <xsd:enumeration value="HOURS"/>
               <xsd:enumeration value="MINUTES"/>
               <xsd:enumeration value="SECONDS"/>
           </xsd:restriction>
       </xsd:simpleType>




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


Reply via email to