[sr-dev] [tracker] Comment added: app_lua: add rpc reload command

[sip-router]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#270 - app_lua: add rpc reload command
User who did this - Daniel-Constantin Mierla (miconda)

----------
It doesn't seem that sr_lua_script_ver->len needs to be protected by a lock, it is set once, outside of any lock, at init. Then you read it inside a lock, but no other possibility to be updated at the same time.

Checking if there is need to reload scripts every time a function is executed seems a bit too time consuming from my point of view. Perhaps there can be added some optimization, but that can be subject to future improvements. For the beginning it would be good to have a module parameter that enables/disables the reloading. It this way one that does not need reloading does not have to be exposed to the loops executed to detect and reload scripts for each function execution.
----------

More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=270#comment797

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.