Beefy Boxes and Bandwidth Generously Provided by pair Networks
P is for Practical

Re^8: Use more threads.

by BrowserUk (Pope)
on Apr 30, 2013 at 11:27 UTC ( #1031391=note: print w/replies, xml ) Need Help??

in reply to Re^7: Use more threads.
in thread Use more threads.

Simple. Don't load everything into every thread.

  1. Tk isn't threadsafe so it has no business being loaded by more than 1 thread -- preferably the main thread -- so don't use it at the top of the code, require your TK::* modules after you've created your pool of threads.
  2. Likewise, it doesn't make much sense to have DBI/DBD modules in multiple threads. The DB server will serialise the accesses from the multiple threads anyway, with the result that queries and stores will be conflicting within the server and slowing each other down. So serialise it at your end. Start a dedicated DBI thread and queue your accesses to it.

Those two steps alone cut the load time by half on my machine:

C:\test>junk rem As your posted it. Took 4.529284 seconds C:\test>junk rem Tk required after threads started. Took 3.111319 seconds C:\test>junk rem DBI/DBD moved to separate thread. Took 2.166010 seconds

And that is (probably) only the tip of the iceberg. With proper sight of your real code, there are probably lots more simple steps that could be done to improve things no end.

With the rise and rise of 'Social' network sites: 'Computers are making people easier to use everyday'
Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
"Science is about questioning the status quo. Questioning authority".
In the absence of evidence, opinion is indistinguishable from prejudice.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1031391]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others pondering the Monastery: (5)
As of 2018-06-18 05:54 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (108 votes). Check out past polls.