Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@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 1f5444f (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, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.