Attention is currently required from: laforge, pespin, lynxis lazus.
keith has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-msc/+/28338
)
Change subject: Refactor smsq_take_next_sms()
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
I'm somewhat skeptical it makes sense to merge
this to master. […]
ACK. Well we don't want to add extra days!
There are things in this patchset that fix some flawed logic and help to get all SMS
delivered. They probably apply regardless of the storage backend. I'm happy to run
this code locally until we are rid of SQL, then I can refactor and submit this
work on top of that.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-msc/+/28338
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I06c418d4919dd8d28c643b7e0a735bc74d66212c
Gerrit-Change-Number: 28338
Gerrit-PatchSet: 1
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 20 Jun 2022 19:56:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment