quartz-mongodb 2.2.0-rc2: checkInErrorHandler — something is missing

[gagarski]

I am trying to update quartz-mongodb in our project. Unfortunately we had to do some monkey-patching around quartz-mongodb code so I took some look into quartz-mongodb code and noticed the following strange things about setting custom checkInErrorHandler:

• MongoDBJobStore#initialize reads quartz-properties into local variable props
  

  quartz-mongodb/src/main/java/com/novemberain/quartz/mongodb/MongoDBJobStore.java

  Line 112 in 9125643

  Properties props = loadProperties(loadHelper);

• These props are passed into MongoStoreAssembler#build (and nowhere else)
  

  quartz-mongodb/src/main/java/com/novemberain/quartz/mongodb/MongoDBJobStore.java

  Line 114 in 9125643

  assembler.build(this, loadHelper, signaler, props);

• Then these props are passed into MongoStoreAssembler#createCheckinExecutor (and nowhere else)
  

  quartz-mongodb/src/main/java/com/novemberain/quartz/mongodb/MongoStoreAssembler.java

  Line 81 in 9125643

  checkinExecutor = createCheckinExecutor(jobStore, loadHelper, quartzProps);

Now to the strange things:

• createCheckinExecutor ignores these props and calls createCheckinTask (without passing props)
  

  quartz-mongodb/src/main/java/com/novemberain/quartz/mongodb/MongoStoreAssembler.java

  Line 87 in 9125643

  return new CheckinExecutor(createCheckinTask(jobStore, loadHelper),

• createCheckinTask decides which error handler to use based on a value returned by MongoDBJobStore#getCheckInErrorHandler
  

  quartz-mongodb/src/main/java/com/novemberain/quartz/mongodb/MongoStoreAssembler.java

  Lines 95 to 100 in 9125643

  	if (jobStore.getCheckInErrorHandler() == null) {
  	// current default, see
  	aClass = KamikazeErrorHandler.class;
  	} else {
  	aClass = loadHelper.loadClass(jobStore.getCheckInErrorHandler());
  	}

The thing is that setCheckInErrorHandler is never called by anyone.

So, properties are being read for no reason and checkInErrorHandler class is based chosen based on a value which is effectively constant. Seems like something here is missing.

[WakeCaine]

This repo is abandoned. I do recommend switching to SQL based Quartz from Spring. We tried to use this connector in our project but there has been too many problems with actual optimization and speed even on single instance. Switching to SQL version resolved it and allowed for correct multithreading speed.

[rfelgent]

@WakeCaine ,

why are you so sure that this repo is abandoned ?

[wakymak]

@WakeCaine ,

why are you so sure that this repo is abandoned ?

Main maintainer/creator stopped contributions and did not refresh official artifact. Bintray is no longer with us. Additionally there are still persisting problems not only with transactions but also cluster mode.
Not saying this didnt have potential but we would need somebody to actually work and improve this and for now SQL version is a lot faster and more stable. I wanted to keep using this but it failed my expectations.

[alebastrov]

just create a new class

package xxx;
public class FixedMongoDBJobStore extends MongoDBJobStore {
    public String getCheckInErrorHandler() {
        return NoOpErrorHandler.class.getCanonicalName();
    }
}

and pass its name to quartz.properties instead of

#org.quartz.jobStore.class=com.novemberain.quartz.mongodb.MongoDBJobStore
org.quartz.jobStore.class=xxx.FixedMongoDBJobStore