Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2023
  
    @michaelandrepearce The code is fine I think. It'll be nice you add some 
unittests (e.g. in QueueControlTest) to verify it works. Also I find that the 
"Group" in method names a bit confusing.
    Would be using MessageGoup instead more clear? I'm not sure about that as 
I'm not a native speaker, just a personal opinion.


---

Reply via email to