Skip to content

Commit d00bd48

Browse files
author
Bogdan Rancichi
committed
keep only channel & delivery tag on message
1 parent 1381d17 commit d00bd48

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

RabbitMq/BatchConsumer.php

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ protected function handleProcessMessage(AMQPMessage $msg, $processFlag)
134134
$this->consumed++;
135135
$this->maybeStopConsumer();
136136
if (!$isRejectedOrReQueued) {
137-
$this->addDeliveryTag($msg);
137+
$this->addMessage($msg);
138138
}
139139

140140
if (!is_null($this->getMemoryLimit()) && $this->isRamAlmostOverloaded()) {
@@ -150,12 +150,12 @@ protected function handleProcessMessage(AMQPMessage $msg, $processFlag)
150150
private function resetBatch($hasExceptions = false)
151151
{
152152
if ($hasExceptions) {
153-
array_map(function(AMQPMessage $msg) {
154-
$msg->delivery_info['channel']->basic_reject($msg->delivery_info['delivery_tag'], true);
153+
array_map(function($message) {
154+
$message['channel']->basic_reject($message['tag'], true);
155155
}, $this->messages);
156156
} else {
157-
array_map(function(AMQPMessage $msg) {
158-
$msg->delivery_info['channel']->basic_ack($msg->delivery_info['delivery_tag']);
157+
array_map(function($message) {
158+
$message['channel']->basic_ack($message['tag']);
159159
}, $this->messages);
160160
}
161161

@@ -168,9 +168,12 @@ private function resetBatch($hasExceptions = false)
168168
*
169169
* @return void
170170
*/
171-
private function addDeliveryTag(AMQPMessage $message)
171+
private function addMessage(AMQPMessage $message)
172172
{
173-
$this->messages[$this->batchCounter++] = $message;
173+
$this->messages[$this->batchCounter++] = array(
174+
'channel' => $message->delivery_info['channel'],
175+
'tag' => $message->delivery_info['delivery_tag'],
176+
);
174177
}
175178

176179
/**

0 commit comments

Comments
 (0)