added comment - I'd have thought a Map was better?

git-svn-id: https://svn.apache.org/repos/asf/incubator/activemq/trunk@381612 13f79535-47bb-0310-9956-ffa450edef68

activemq-core/src/main/java/org/apache/activemq/ActiveMQSessionExecutor.java

@@ -86,6 +86,9 @@ public class ActiveMQSessionExecutor implements Task {
     }
     
     void dispatch(MessageDispatch message){
+
+        // TODO  - we should use a Map for this indexed by consumerId
+        
         for (Iterator i = this.session.consumers.iterator(); i.hasNext();) {
             ActiveMQMessageConsumer consumer = (ActiveMQMessageConsumer) i.next();
             ConsumerId consumerId = message.getConsumerId();