From dbe72fe26dfde895cea17fe4e007b3035f6f2e99 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Hanol?= <regis@hanol.fr>
Date: Fri, 14 Mar 2014 15:53:58 +0100
Subject: [PATCH] FEATURE: minimize duration of read-only mode when backing up

---
 lib/export/exporter.rb | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/lib/export/exporter.rb b/lib/export/exporter.rb
index 6b2de492aa..59f1eeeed2 100644
--- a/lib/export/exporter.rb
+++ b/lib/export/exporter.rb
@@ -19,22 +19,25 @@ module Export
 
       listen_for_shutdown_signal
 
+      ensure_directory_exists(@tmp_directory)
+      ensure_directory_exists(@archive_directory)
+
+      write_metadata
+
+      ### READ-ONLY / START ###
       enable_readonly_mode
 
       pause_sidekiq
       wait_for_sidekiq
 
-      ensure_directory_exists(@tmp_directory)
-
-      write_metadata
-
       dump_public_schema
 
-      update_dump
+      disable_readonly_mode
+      ### READ-ONLY / END ###
 
       log "Finalizing backup..."
 
-      ensure_directory_exists(@archive_directory)
+      update_dump
 
       create_archive
 
@@ -268,7 +271,7 @@ module Export
       log "Cleaning stuff up..."
       remove_tmp_directory
       unpause_sidekiq
-      disable_readonly_mode
+      disable_readonly_mode if Discourse.readonly_mode?
       mark_export_as_not_running
       log "Finished!"
     end