Job Scheduler

Improvements for logging on order.setback()

Details

  • Type: Improvement Improvement
  • Status: Accepted Accepted
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.3.1, 1.3.2, 1.3.3
  • Fix Version/s: next
  • Component/s: Job Scheduler Binaries
  • Description:
    Hide

    Change log message "SCHEDULER-947 setback(): order has been set back..." from warning to info.
    This warning doesn't give any information why the order was setback and may cause confusion if no setback is configured.
    It is better if the job implementation creates a warning when the order can no longer be set back. But the job implementation has to know when an order could not be set back anymore (because max setback has been reached).
    Therefore order.setback() needs to return a boolean value:
    true - the order was set back and will be processed again later
    false - the order could not be set back any more

    the common usage would be:
    if (!order.setback()) spooler_log.warn("Failed to find...");

    Show
    Change log message "SCHEDULER-947 setback(): order has been set back..." from warning to info. This warning doesn't give any information why the order was setback and may cause confusion if no setback is configured. It is better if the job implementation creates a warning when the order can no longer be set back. But the job implementation has to know when an order could not be set back anymore (because max setback has been reached). Therefore order.setback() needs to return a boolean value: true - the order was set back and will be processed again later false - the order could not be set back any more the common usage would be: if (!order.setback()) spooler_log.warn("Failed to find...");

Activity

There are no comments yet on this issue.

People

Dates

  • Created:
    27 August 2007 18:07
    Updated:
    18 November 2011 21:32