Harden worker and improve script caching #90

Merged
merged 3 commits into from Apr 5, 2013

Conversation

Projects
None yet
1 participant
Contributor

myronmarston commented Apr 5, 2013

@proby -- want to review this?

myronmarston added some commits Apr 5, 2013

Remove the lua script cache.
(I'm going to implement a better solution next).
Only load lua scripts if they are needed.
Given how many workers we are and how rarely the scripts change,
most of the time the lua scripts do not need to be loaded when
a client is created.  It can be delayed until it is actually
needed (often never during the lifetime of the process).

myronmarston added a commit that referenced this pull request Apr 5, 2013

Merge pull request #90 from seomoz/harden_worker
Harden worker and improve script caching

@myronmarston myronmarston merged commit 7502f0e into master Apr 5, 2013

1 check passed

default The Travis build passed
Details
Contributor

myronmarston commented Apr 5, 2013

I merged this since @benkirzhner verbally LGTM'd it.

@myronmarston myronmarston deleted the harden_worker branch Apr 5, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment