Skip to content

Commit 9dd986c

Browse files
author
freyhsiao
committed
add comments.
1 parent 6c6c04b commit 9dd986c

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

src/Queue/CMQQueue.php

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,13 @@ class CMQQueue extends Queue implements QueueContract
4040
*/
4141
private static $createPayload;
4242

43+
/**
44+
* CMQQueue constructor.
45+
* @param Account $queueAccount
46+
* @param Account $topicAccount
47+
* @param array $config
48+
* @throws \ReflectionException
49+
*/
4350
public function __construct(Account $queueAccount, Account $topicAccount, array $config)
4451
{
4552
$this->queueAccount = $queueAccount;
@@ -129,12 +136,12 @@ public function pushRaw($payload, $queue = null, array $options = [])
129136
if ($driver instanceof Topic) {
130137
switch ($this->topicOptions['filter']) {
131138
case self::CMQ_TOPIC_TAG_FILTER_NAME:
132-
return retry(Arr::get($this->topicOptions, 'retries', 1),
139+
return retry(Arr::get($this->topicOptions, 'retries', 3),
133140
function () use ($driver, $message, $queue) {
134141
return $driver->publish_message($message->msgBody, explode(',', $queue), null);
135142
});
136143
case self::CMQ_TOPIC_ROUTING_FILTER_NAME:
137-
return retry(Arr::get($this->topicOptions, 'retries', 1),
144+
return retry(Arr::get($this->topicOptions, 'retries', 3),
138145
function () use ($driver, $message, $queue) {
139146
$driver->publish_message($message->msgBody, [], $queue);
140147
});
@@ -145,7 +152,7 @@ function () use ($driver, $message, $queue) {
145152
}
146153
}
147154

148-
return retry(Arr::get($this->queueOptions, 'retries', 1), function () use ($driver, $message, $options) {
155+
return retry(Arr::get($this->queueOptions, 'retries', 3), function () use ($driver, $message, $options) {
149156
return $driver->send_message($message, Arr::get($options, 'delay', 0));
150157
});
151158
}

tests/CMQQueueTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,14 @@ public function provider()
2929
'host' => env('CMQ_QUEUE_HOST', 'https://cmq-queue-region.api.qcloud.com'),
3030
'name' => env('CMQ_QUEUE', 'default'),
3131
'polling_wait_seconds' => env('CMQ_QUEUE_POLLING_WAIT_SECONDS', 0), //0-30seconds
32-
'retries' => env('CMQ_QUEUE_RETRIES', 1),
32+
'retries' => env('CMQ_QUEUE_RETRIES', 3),
3333
],
3434
'topic' => [
3535
'enable' => env('CMQ_TOPIC_ENABLE', false),
3636
'filter' => env('CMQ_TOPIC_FILTER', 'routing'), //routing or msgtag
3737
'host' => env('CMQ_TOPIC_HOST', 'https://cmq-topic-region.api.qcloud.com'),
3838
'name' => env('CMQ_TOPIC'),
39-
'retries' => env('CMQ_TOPIC_RETRIES', 1),
39+
'retries' => env('CMQ_TOPIC_RETRIES', 3),
4040
],
4141
],
4242

0 commit comments

Comments
 (0)