Attention is currently required from: keith. pespin has posted comments on this change by keith. ( https://gerrit.osmocom.org/c/osmo-msc/+/28342?usp=email )
Change subject: SMS UX optimisation: Deliver rx'd SMS right away ...................................................................... Patch Set 4: (1 comment) Patchset: PS4: I'm not really involved into the big picture/roadmap regarding SMS, but I'd argue it makes more sense to send SMS in order in the queue instead of reordering SMS (what would happen if we send the SMS directly upon receive). Think for example 2 SMS sent by Alice to Bob where the 2nd SMS provides contradictory information wrt 1st SMS (eg. "I'm arriving on time" vs "I'm late", or eg. 2-factor authentication codes). Without looking in detail, may it be that the delay @[email protected] is mentioning comes from the fact that when we run the queue we try to lookup for SMS for all subscribers and retrigger them? AFAIU the sane approach would be to lookup SMS for that specific subscriber, not sure if we are doing that already. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/28342?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I9af51ef0d9c2e6c5acc5128efd6195df881b680c Gerrit-Change-Number: 28342 Gerrit-PatchSet: 4 Gerrit-Owner: keith <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <[email protected]> Gerrit-CC: neels <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: keith <[email protected]> Gerrit-Comment-Date: Tue, 04 Nov 2025 11:10:30 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
