[jboss-jira] [JBoss JIRA] Closed: (JBMESSAGING-1351) Revisit flush on ClientConsumerImpl

Revisit flush on ClientConsumerImpl ----------------------------------- Key: JBMESSAGING-1351 URL: http://jira.jboss.com/jira/browse/JBMESSAGING-1351 Project: JBoss Messaging Issue Type: Bug Affects Versions: 2.0.0 Alpha Reporter: Clebert Suconic Assigned To: Tim Fox Fix For: 2.0.0 Beta I have fixed AcknowledgementTest right before the Alpha release. I didn't want to change semantics of callOnMessage at this point... that's why I crated this task. When the MessageListener is set after the messages were already received on the consumer, there is nothing checking for messages already received and flushing the buffer. handleMessage verifies for the handler and will send messages to the Receiver right away.. aways removing only one message from the buffer. IMO callOnMessage should be aways in loop while the buffer has available messages, but as I said for alpha I'm doing the loop only when setMessageListener is called. This fix needs revisiting.