Fixing a recoverable journal should not result in the zone being written [v9.16]
requested to merge 2623-9-16-13-overwrites-master-files-if-old-format-jnl-files-are-present-v9_16 into v9_16
when dns_journal_rollforward returned ISC_R_RECOVERABLE the distintion between 'up to date' and 'success' was lost, as a consequence zone_needdump() was called writing out the zone file when it shouldn't have been. This change restores that distintion. Adjust system test to reflect visible changes.
(cherry picked from commit ec7a9af3)
Closes #2623 (closed)