Continuum

Once CRON Expression Has Error, Server Never Startup

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.0.2
  • Fix Version/s: 1.0.3
  • Component/s: None
  • Labels:
    None
  • Environment:
    SuSE Linux 9.2 AMD64
  • Complexity:
    Intermediate
  • Number of attachments :
    1

Description

To reproduce the problem:

1. Add a schedule with the following CRON expression:
0 0 4-1 * * ?
2. Restart the Continuum server.
3. The server never start up.

To workaround the problem:

1. Use JDBC to dig int he database and fix the CRON expression:
0 0 4-23 * * ?
2. Start the Continuum server.
3. Server starts successfully.

Issue Links

Activity

Hide
Emmanuel Venisse added a comment -

Arghh, the cron expression validator from quartz is bad. I'll add more test in our own validator and will check that ranges are in correct order.

Show
Emmanuel Venisse added a comment - Arghh, the cron expression validator from quartz is bad. I'll add more test in our own validator and will check that ranges are in correct order.
Hide
Richard C. L. Li added a comment -

Besides fixing the validator, I think codes in application startup should be fixed as well so that the application can staert up normally with invalid schedule disabled. Otherwise it is too hard for an administrator to fix the problem, since the broken schedule will completely bring down the web application and need to direct alter the databaase to change the schedule.

Show
Richard C. L. Li added a comment - Besides fixing the validator, I think codes in application startup should be fixed as well so that the application can staert up normally with invalid schedule disabled. Otherwise it is too hard for an administrator to fix the problem, since the broken schedule will completely bring down the web application and need to direct alter the databaase to change the schedule.
Hide
Emmanuel Venisse added a comment -

sure.
Can you attach stacktrace when continnuum can't start with a bad cron expression?

Show
Emmanuel Venisse added a comment - sure. Can you attach stacktrace when continnuum can't start with a bad cron expression?
Hide
Richard C. L. Li added a comment -

Stack trace for server start failed.

Show
Richard C. L. Li added a comment - Stack trace for server start failed.
Hide
Emmanuel Venisse added a comment -

Fixed.
I'll close this issue when we'll use the new cron expression validator.

Show
Emmanuel Venisse added a comment - Fixed. I'll close this issue when we'll use the new cron expression validator.
Hide
Emmanuel Venisse added a comment -

Done.

Show
Emmanuel Venisse added a comment - Done.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: