Not that familiar with Slide internals ;)
But when I had to find the threads and change in our framework I
searched for "new Thread", "extends Thread" and "implements Runnable"
for example :)
Cheers
Eiki
On Apr 12, 2007, at 5:52 AM, Oliver Zeigermann wrote:
You are certainly right. Do you know of any other worker threads?
Oliver
2007/4/11, Eirikur Hrafnsson <[EMAIL PROTECTED]>:
A quick note on thread usage like that in Slide, shouldn't those
threads be daemon threads?
I ask because sometimes our apps don't shutdown using shutdown.sh
unless we add the -force parameter could be related to Slide since we
have changed all our threading to daemons so no matter what they are
killable...
-Eiki
p.s.
Anyone have a clue how to solve my TooManyClauses problem I posted
earlier?
On Apr 11, 2007, at 3:59 PM, [EMAIL PROTECTED] wrote:
> Author: ozeigermann
> Date: Wed Apr 11 08:59:18 2007
> New Revision: 527546
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=527546
> Log:
> Quick hack to prevent spawning of innumerous threads when there is
> no correct check for a connected store
>
> Modified:
> jakarta/slide/trunk/src/stores/org/apache/slide/index/lucene/
> Index.java
>
> Modified: jakarta/slide/trunk/src/stores/org/apache/slide/index/
> lucene/Index.java
> URL: http://svn.apache.org/viewvc/jakarta/slide/trunk/src/stores/
> org/apache/slide/index/lucene/Index.java?
> view=diff&rev=527546&r1=527545&r2=527546
>
=====================================================================
=
> ========
> --- jakarta/slide/trunk/src/stores/org/apache/slide/index/lucene/
> Index.java (original)
> +++ jakarta/slide/trunk/src/stores/org/apache/slide/index/lucene/
> Index.java Wed Apr 11 08:59:18 2007
> @@ -171,11 +171,19 @@
>
> public void start() {
> if (configuration.indexAsynchron) {
> - this.indexThread = new JobRunner();
> - this.indexThread
> - .setName("Indexing Thread
(" + this.indexName + ")");
> - this.indexThread.setPriority
(configuration.getPriority());
> - this.indexThread.start();
> + if (this.indexThread != null) {
> + // this really should be WARNING, but this can get
> called so
> + // many times
> +/* getLogger().log(
> + "Indexing thread has already been started!
> Not starting a new one!",
> + Logger.WARNING);
> +*/
> + } else {
> + this.indexThread = new JobRunner();
> + this.indexThread.setName("Indexing Thread (" +
> this.indexName + ")");
> + this.indexThread.setPriority
> (configuration.getPriority());
> + this.indexThread.start();
> + }
> }
> }
>
>
>
>
>
---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
