@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/[email protected]>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to [email protected]

Reply via email to