FishEye: history quartz/trunk/src/java/org/quartz/simpl/SimpleThreadPool.java

824 annotated / raw | Diffs: previous, other | Lines: 576 ( +23, -12 )

Created: 2008-08-19 23:27:54 -0500 (47 hours ago) | Author: jhouse | Changeset: 824

QUARTZ-668, QUARTZ-669: fixes to threading issues with multicore/multicpu systems.

811 annotated / raw | Diffs: previous, other | Lines: 565 ( +0, -7 )

Created: 2008-06-20 08:38:51 -0500 (2 months 1 day ago) | Author: jhouse | Changeset: 811

QUARTZ-674: Superfluous info logged in SimpleThreadPool shutdown()

658 annotated / raw | Diffs: previous, other | Lines: 572 ( +11, -9 )

Created: 2007-03-10 07:02:55 -0600 (17 months ago) | Author: perfnorm | Changeset: 658

Fixed checkstyle warnings

612 annotated / raw | Diffs: previous, other | Lines: 570 ( +111, -141 )

Created: 2006-08-27 11:36:42 -0500 (23 months ago) | Author: jhouse | Changeset: 612

general overhaul of locking, thread-handoff within SimpleThreadPool

Branch point for: quartz_1-6

Tags: quartz_1-6-0-alpha

604 annotated / raw | Diffs: previous, other | Lines: 600 ( +1, -2 )

Created: 2006-07-28 07:57:21 -0500 (2 years ago) | Author: jrosenberg | Changeset: 604

Issue number: Checkstyle
Fixed misplaced curly brace

603 annotated / raw | Diffs: previous, other | Lines: 601 ( +17, -19 )

Created: 2006-07-27 09:01:34 -0500 (2 years ago) | Author: jhouse | Changeset: 603

QUARTZ-491 - fix synchronization of availCount

597 annotated / raw | Diffs: previous, other | Lines: 603 ( +9, -0 )

Created: 2006-07-24 11:02:15 -0500 (2 years ago) | Author: jhouse | Changeset: 597

QUARTZ-491

595 annotated / raw | Diffs: previous, other | Lines: 594 ( +1, -1 )

Created: 2006-07-18 08:40:20 -0500 (2 years 1 month ago) | Author: jrosenberg | Changeset: 595

Issue number: QUARTZ-13
Fixed off by one issue with count of available threads

574 annotated / raw | Diffs: previous, other | Lines: 594 ( +5, -3 )

Created: 2006-07-05 08:51:14 -0500 (2 years 1 month ago) | Author: jrosenberg | Changeset: 574

Issue number: QUARTZ-13
Fixed checkstyle issues.

570 annotated / raw | Diffs: previous, other | Lines: 592 ( +49, -26 )

Created: 2006-06-21 18:16:39 -0500 (2 years 2 months ago) | Author: jhouse | Changeset: 570

QUARTZ-13 - don't acquire trigger until thread is available.

528 annotated / raw | Diffs: previous, other | Lines: 569 ( +3, -0 )

Created: 2006-05-20 10:27:33 -0500 (2 years 3 months ago) | Author: jrosenberg | Changeset: 528

Issue number: QUARTZ-450
If "org.quartz.threadPool.makeThreadsDaemons" is true and "org.quartz.threadPool.threadsInheritGroupOfInitializingThread" is false, then mark the new thread group as daemon.

426 annotated / raw | Diffs: previous, other | Lines: 566 ( +43, -32 )

Created: 2006-03-20 15:40:36 -0600 (2 years 5 months ago) | Author: jrosenberg | Changeset: 426

Issue number: QUARTZ-374
Fixed main Quartz src to comply with checkstyle

413 annotated / raw | Diffs: previous, other | Lines: 555 ( +3, -1 )

Created: 2006-03-16 11:03:27 -0600 (2 years 5 months ago) | Author: jrosenberg | Changeset: 413

Issue number: QUARTZ-369
Cleaned up the logging to be consistent, following these rules:
1. Do not cache the log statically, to avoid potential redeployment issues.
2. Where appropriate, do cache the log in a private final member. By appropriate, I mean if the class always uses the log during normal execution, or if very few of the object are ever created.
For example, AttributeRestoringConnectionWrapper does not cache the log because it only uses the log for warning cases, and an instance is created per connection.
private final Log log = LogFactory.getLog(getClass());
3. Expose the log through a protected non-static getLog() method that can be then used by any subclass (existing public methods are left public for backwards compatibility, though in a few cases a static getLog() method was changed to be non-static which would break backwards compatibility if the methods were being used, though it seems extremely unlikely they would be.)
4. Pass getClass() to LogFactory.getLog() rather than ClassName.class, so that subclasses always get the correct category name
5. One exception to these rules is UserTransactionHelper which is an all static class, and so has a static getLog method, does not cache the log, and must use the UserTransactionHelper.class category.
6. Also did a quick run through to make sure any debug messages that are building strings dynamically are protected by isDebugEnabled()

250 annotated / raw | Diffs: previous, other | Lines: 553 ( +14, -3 )

Created: 2005-09-23 21:02:56 -0500 (2 years 10 months ago) | Author: jhouse | Changeset: 250

just some minor refactoring of the creation of the threads.
CVS: ----------------------------------------------------------------------
CVS: Issue number:
CVS:   If this change addresses one or more issues,
CVS:   then enter the issue number(s) here.
CVS: Obtained from:
CVS:   If this change has been taken from another system,
CVS:   then name the system in this line, otherwise delete it.
CVS: Submitted by:
CVS:   If this code has been contributed to the project by someone else; i.e.,
CVS:   they sent us a patch or a set of diffs, then include their name/email
CVS:   address here. If this is your work then delete this line.
CVS: Reviewed by:
CVS:   If we are doing pre-commit code reviews and someone else has
CVS:   reviewed your changes, include their name(s) here.
CVS:   If you have not had it reviewed then delete this line.

Branch point for: b_quartz_1-5-x

Tags: v1_5_0 quartz_1-5-1 quartz_1-5-2

217 annotated / raw | Diffs: previous, other | Lines: 542 ( +0, -1 )

Created: 2005-08-09 23:31:42 -0500 (3 years ago) | Author: jhouse | Changeset: 217

doh! can't change daemon state of thread.
CVS: ----------------------------------------------------------------------
CVS: Issue number:
CVS:   If this change addresses one or more issues,
CVS:   then enter the issue number(s) here.
CVS: Obtained from:
CVS:   If this change has been taken from another system,
CVS:   then name the system in this line, otherwise delete it.
CVS: Submitted by:
CVS:   If this code has been contributed to the project by someone else; i.e.,
CVS:   they sent us a patch or a set of diffs, then include their name/email
CVS:   address here. If this is your work then delete this line.
CVS: Reviewed by:
CVS:   If we are doing pre-commit code reviews and someone else has
CVS:   reviewed your changes, include their name(s) here.
CVS:   If you have not had it reviewed then delete this line.

Tags: quartz_1_5_0_rc1

190 annotated / raw | Diffs: previous, other | Lines: 543 ( +4, -1 )

Created: 2005-08-06 12:05:46 -0500 (3 years ago) | Author: jhouse | Changeset: 190

fix the thread back up if the runnable changed its settings.
CVS: ----------------------------------------------------------------------
CVS: Issue number:
CVS:   If this change addresses one or more issues,
CVS:   then enter the issue number(s) here.
CVS: Obtained from:
CVS:   If this change has been taken from another system,
CVS:   then name the system in this line, otherwise delete it.
CVS: Submitted by:
CVS:   If this code has been contributed to the project by someone else; i.e.,
CVS:   they sent us a patch or a set of diffs, then include their name/email
CVS:   address here. If this is your work then delete this line.
CVS: Reviewed by:
CVS:   If we are doing pre-commit code reviews and someone else has
CVS:   reviewed your changes, include their name(s) here.
CVS:   If you have not had it reviewed then delete this line.

145 new annotated / raw | Lines: 540 ( +540, -0 )

Created: 2005-07-12 15:28:48 -0500 (3 years 1 month ago) | Author: jhouse | Changeset: 145

new files (reorganizing repository)
CVS: ----------------------------------------------------------------------
CVS: Issue number:
CVS:   If this change addresses one or more issues,
CVS:   then enter the issue number(s) here.
CVS: Obtained from:
CVS:   If this change has been taken from another system,
CVS:   then name the system in this line, otherwise delete it.
CVS: Submitted by:
CVS:   If this code has been contributed to the project by someone else; i.e.,
CVS:   they sent us a patch or a set of diffs, then include their name/email
CVS:   address here. If this is your work then delete this line.
CVS: Reviewed by:
CVS:   If we are doing pre-commit code reviews and someone else has
CVS:   reviewed your changes, include their name(s) here.
CVS:   If you have not had it reviewed then delete this line.

FishEye

Line History

Show Arbitrary Diff

History

latest revision Branch trunk

latest revision Branch b_quartz_1-5-x

latest revision Branch quartz_1-6