23 Dec
2023
23 Dec
'23
12:58 a.m.
@xkaraman: thanks for looking into #3064 and proposing a PR!
As I looked at the commit of this PR and reviewed the code, the purpose was to avoid doing
lookup of the event route every time, but do it only once per process. The commit in this
PR resets the global variable so lookup is done again and should make a fix for #3064, but
still doing the lookup every time.
I pushed an alternative commit to master branch that does the caching of the first lookup
in the global variable, so it is going to be done only once, avoiding the search in the
hash table of event route blocks each time.
Should commit 1f5444fd2e25d227ffd70f1087d057a3b8002558 (now in master) fail to fix #3064,
then we can look further into it.
This PR should be closed if you think the above referenced commit fixes it.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3687#issuecomment-1867718763
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3687/c1867718763(a)github.com>