Quartz starts unmanaged threads in JobStoreSupport

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: 2 Major 2 Major
  • Resolution: Fixed
  • Affects Version/s: 1.8.5, 2.0.2
  • Fix Version/s: 2.1
  • Component/s: Core, ThreadPools
  • Labels:
    None
  • Terracotta Target:
    Quartz 2.1.0
  • Fixed In Revision:
    1633

Description

This is a followup to QTZ-113. QuartzSchedulerThread is now started using the new ThreadExecutor. But there are two other threads that are started in their initialize() method:

JobStoreSupport$MisfireHandler
JobStoreSupport$ClusterManager

They should probably also be started using ThreadExecutor.

Issue Links

clones

Bug - A problem which impairs or prevents the functions of the product. QTZ-233 CLONE -Quartz starts unmanaged threads in JobStoreSupport

  • 2 Major - Fix for next major release.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Bug - A problem which impairs or prevents the functions of the product. QTZ-58 Authorization problems for secure jobs

  • 2 Major - Fix for next major release.
  • Open - The issue is open and ready for the assignee to start work on it.
relates to

Bug - A problem which impairs or prevents the functions of the product. QTZ-113 Quartz starts unmanaged threads and J2EE context is not propagated to the job threads

  • 1 Critical - Fix for next maintenance release (double-dot release).
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
James House added a comment -

Would like to do this in 2.0.3 as long as it can be done without an interface (JobStore) change. Otherwise defer to 2.1

Show
James House added a comment - Would like to do this in 2.0.3 as long as it can be done without an interface (JobStore) change. Otherwise defer to 2.1
Hide
Permalink
Zemian Deng added a comment -

I can look into this.

Show
Zemian Deng added a comment - I can look into this.
Hide
Permalink
James House added a comment -

Thanks!

Show
James House added a comment - Thanks!
Hide
Permalink
Zemian Deng added a comment -

This is fixed on 2.0.x branch. No significant API changes are introduced.

Same changes are merged to trunk on r1634 as well.

Show
Zemian Deng added a comment - This is fixed on 2.0.x branch. No significant API changes are introduced. Same changes are merged to trunk on r1634 as well.

People

  • Assignee:
    Zemian Deng
    Reporter:
    Stefan Gybas
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: