Attention is currently required from: keith, pespin.

fixeria has posted comments on this change by keith. ( https://gerrit.osmocom.org/c/osmo-msc/+/28340?usp=email )

Change subject: sqlite optimisation: Avoid unneeded database operation ......................................................................

Patch Set 7: Code-Review+1

(1 comment)

File src/libmsc/db.c:

https://gerrit.osmocom.org/c/osmo-msc/+/28340/comment/3398893b_f0566a9e?usp=... : PS4, Line 303: DB_STMT_SMS_DEL_EXPIRED

I'd prefer to keep you happy in this patch. I can remove the LIMIT clause.

Thanks.

I'd propose to get rid of this one (EXPIRED) an replace its use in the code with the other one, just plain old: DB_STMT_SMS_DEL_BY_ID

Ack, this is exactly what I had in mind.