izb/Swiffotron — Issue #79: storeWriteEnabled means swfout keys are redundant and mandatory
Author: izb
Created: 2011-10-18 10:38:45+00:00
State: open
Labels: low-priority, missing-unit-test

For some setups, we disable all writing to the store with storeWriteEnabled = false in the swiffotron config.

If this is the case, then the swfout store keys are redundant in the job.xml files. They are also mandatory. Omitting the swfout tag means Swiffotron decides no work should be done.

We should re-think this carefully. Two obvious fixes are either to make store keys optional on swfout tags. This may replace the storeWriteEnabled flag.

Alternatively we may wish to determine that works needs done by the presence of cache keys as well as swfout tags, making the swfout tag optional on job primary outputs. Note - this might actually be the case now, I'm not sure. Need to set up a new unit test to check this.