Set Sensible Default for Workspace Cleanup Period

Steps to reproduce

The default value is set to -1 which will break the quartz scheduler as shown in the unit tests:

01:35:20 ERROR (WorkspaceCleaner:100) - Error scheduling Quartz job
java.lang.IllegalArgumentException: Repeat interval must be >= 0
at org.quartz.SimpleTrigger.setRepeatInterval(SimpleTrigger.java:471)
at org.quartz.TriggerUtils.makeSecondlyTrigger(TriggerUtils.java:504)
at org.quartz.TriggerUtils.makeSecondlyTrigger(TriggerUtils.java:481)
at org.opencastproject.workspace.impl.WorkspaceCleaner.schedule(WorkspaceCleaner.java:88)
at org.opencastproject.workspace.impl.WorkspaceImpl.activate(WorkspaceImpl.java:198)
at org.opencastproject.workspace.impl.WorkspaceImplTest.setUp(WorkspaceImplTest.java:56)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:483)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:283)
at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:173)
at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:128)
at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203)
at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)

Assignee

Lars Kiesow

Reporter

Lars Kiesow

Severity

Incorrectly Functioning With Workaround

Tags (folksonomy)

None

Components

Fix versions

Priority

Minor
Configure