[Asterisk-code-review] res_config_pgsql: Limit realtime_pgsql() to return one (no more) record. (asterisk[master])

[George Joseph]

George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/15446 )

Change subject: res_config_pgsql: Limit realtime_pgsql() to return one (no more) record.
......................................................................


Patch Set 2: Code-Review+1

What was the behavior if more than 1 row was returned before this patch?  It looks like we were adding duplicate variables to the list.  If so, that was probably not a good thing so I'd make this a bug fix, not an improvement.  In that case, you could cherry-pick this to the 16 and 18 branches and change the ASTERISK issue to a bug.


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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: If5a6d4b1072ea2e6e89059b21139d554a74b34f5
Gerrit-Change-Number: 15446
Gerrit-PatchSet: 2
Gerrit-Owner: Boris P. Korzun <drtr0jan at yandex.ru>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Comment-Date: Tue, 23 Feb 2021 15:47:52 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210223/055f5ee0/attachment.html>