/**
* Constructs the scheduler.
- * @param aCrawler The interface by which the crawler is executed.
+ * The crawler will run if it is triggered in the range between the minimum (included)
+ * and maximum (included) hour of the day if either
+ * <ul>
+ * <li>it is triggered for the first time on the current day.</li>
+ * <li>an earlier crawling attempt on the same day failed. </li>
+ * </ul>
+ * @param aCrawler The interface through which the crawler is executed.
* @param aHourMin The crawler may only run if hour >= <code>aHourMin</code>
* @param aHourMax The crawler may only run if hour <= <code>aHourMax</code>
*/
public CrawlerSchedule(CrawlerExecutor aCrawler, int aHourMin, int aHourMax) {
_crawler = aCrawler;
- _lastExecuted = null;
- _lastResult = false;
+ _lastExecuted = new Date();
+ _lastResult = true; // the crawler will automatically run the next day.
_lastException = null;
_hourMin = aHourMin;
_hourMax = aHourMax;
* @param aDate Time at which we are executing now.
*/
public void execute(Date aDate) {
+
if (mustExecute(aDate)) {
LOG.info("Executing crawler at " + aDate);
try {
return _lastExecuted;
}
+ public void setLastExecuted(Date aDate) {
+ _lastExecuted = aDate;
+ }
+
/**
* Gets the result of the last execution.
* @return True iff last execution was a success.
* @return True iff the crawler must be run.
*/
private boolean mustExecute(Date aDate) {
+ if ( _lastExecuted == null ) {
+ return false; // crawler must be started manually at least once after deployment.
+ }
Calendar calendar = Calendar.getInstance();
calendar.setTime(aDate);
int hour = calendar.get(Calendar.HOUR_OF_DAY);
if ( hour < _hourMin ) {
- return false;
+ return false;
}
if (hour > _hourMax ) {
return false;
return false; // already run successfully today.
}
+ /**
+ * Determines if the last execution was on the same day.
+ * @param aDate Current time.
+ * @return True iff last execution was on the same day.
+ */
private boolean lastExecutionWasOnSameDay(Date aDate) {
if ( _lastExecuted == null ) {
return false;
}
int curDay = getDayOfYear(aDate);
int lastDay = getDayOfYear(_lastExecuted);
- return curDay == lastDay;
+ return curDay == lastDay; // check can be invalid only if scheduling interval is one year,
+ // which is ridiculous.
}
/**
- * @param aDate
+ * Gets the day of the year
+ * @param aDate Date to compute day for.
*/
private int getDayOfYear(Date aDate) {
Calendar calendar = Calendar.getInstance();