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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: lynxis lazus <[email protected]>
Gerrit-Comment-Date: Mon, 20 Jun 2022 19:56:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment

Reply via email to