michaelandrepearce commented on a change in pull request #3564:
URL: https://github.com/apache/activemq-artemis/pull/3564#discussion_r627172760



##########
File path: 
examples/features/clustered/clustered-jgroups/src/main/resources/activemq/server1/broker.xml
##########
@@ -42,9 +42,9 @@ under the License.
 
       <broadcast-groups>
          <broadcast-group name="my-broadcast-group">
-            <broadcast-period>5000</broadcast-period>
             <jgroups-file>test-jgroups-file_ping.xml</jgroups-file>
             <jgroups-channel>active_broadcast_channel</jgroups-channel>
+            <broadcast-period>5000</broadcast-period>

Review comment:
       Depends on the test, if its a very small unit within the module itself. 
But in general theres actually a tests module where tests are divided into 
type, which will run with different maven profiles. 




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