[Asterisk-code-review] media_cache: Don't lock when curl the remote file (asterisk[master])

George Joseph asteriskteam at digium.com
Thu Aug 19 07:23:01 CDT 2021


Attention is currently required from: Joshua Colp, Andre Barbosa.
George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/16233 )

Change subject: media_cache: Don't lock when curl the remote file
......................................................................


Patch Set 4: Code-Review-1

(1 comment)

Commit Message:

https://gerrit.asterisk.org/c/asterisk/+/16233/comment/b1f35364_6a03c2c7 
PS4, Line 1: Parent:     b5709e61 (cdr_adaptive_odbc: Prevent filter warnings)
Can you re-wrap the commit message at 72 characters?
Also in future, topics should be the Jira issue ID
like "ASTEWRISK-29544".  I fixed that for you.
You can cherry-pick this to the 16, 18 and 19 branches
after you re-wrap.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16233
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I8d4142b463ae4a1d4c41bff2bf63324821567408
Gerrit-Change-Number: 16233
Gerrit-PatchSet: 4
Gerrit-Owner: Andre Barbosa <andre.emanuel.barbosa at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Andre Barbosa <andre.emanuel.barbosa at gmail.com>
Gerrit-Comment-Date: Thu, 19 Aug 2021 12:23:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210819/6fe5d0b4/attachment.html>


More information about the asterisk-code-review mailing list