Bug 229146 - [dstore][multithread] miner threads are not terminated
Summary: [dstore][multithread] miner threads are not terminated
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.0 M7   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-04-28 15:09 EDT by Noriaki Takatsu CLA
Modified: 2011-05-25 07:32 EDT (History)
2 users (show)

See Also:


Attachments
patch to terminate miner threads (1.13 KB, patch)
2008-04-28 15:20 EDT, Noriaki Takatsu CLA
mober.at+eclipse: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Noriaki Takatsu CLA 2008-04-28 15:09:52 EDT
Miner threads are not terminated even after clients disconnect the server
in multithread environment.
This is a problem in multithread server (a single-process server).
Comment 1 Noriaki Takatsu CLA 2008-04-28 15:20:48 EDT
Created attachment 97826 [details]
patch to terminate miner threads

Legal Message:
 I, {Noriaki Takatsu}, declare that I developed attached code from scratch, 
 without referencing any 3rd party materials except material licensed under 
 the EPL. 
 I am authorized by my employer to make this contribution under the EPL.
Comment 2 David McKnight CLA 2008-04-29 10:33:19 EDT
I've committed the fix to cvs.
Comment 3 Martin Oberhuber CLA 2008-04-29 14:18:26 EDT
Please, PLEASE, don't forget setting the "contributed" keyword for contributed patches as soon as they are committed to CVS.

It's part of our IP due diligence to keep track of any non-committer contributions. Because it's so important, we're doing the "double bookkeeping" in both the tm-log.csv and the bugzilla contributed keyword. In the end, a bugzilla query needs to return the same list of contributions as the tm-log.csv.

I'm trying to keep an eye on this, but I also cannot review each and every contribution. Please be diligent yourselves and do BOTH adding the keyword AND updating the tm-log.csv when committing a contributed patch.

We need this little extra effort. 
Thanks.
Comment 4 Martin Oberhuber CLA 2008-04-29 14:25:00 EDT
In order to make sure that everybody understands the duties of super.finish(), I added following Javadoc to the Miner.finish() code:

	 * <p>
	 * Subclasses may override this method but must call
	 * <code>super.finish()</code> at the end of their overriding method.

We should be adding such a paragraph wherever we expect subclasses not to override a method, but rather to extend it. See also 

    AbstractService#initService()
    AbstractService#uninitService()

for example. Please remember adding such comments wherever a super() call is required or expected by the declaring method to be overridden.