From e0a7ced2562f6b65e1db63d7ffdff6d1bf3e8e96 Mon Sep 17 00:00:00 2001
From: Markus Blatt <markus@dr-blatt.de>
Date: Mon, 19 Jun 2017 17:47:24 +0200
Subject: [PATCH] Only create step_timing.txt file if it gets written to.

If output=false is requested we created an empty file.
Now that file will only be created if output was requested
and it will only be opened on one rank in a parallel run.
---
 opm/autodiff/SimulatorBase_impl.hpp              | 12 +++++++-----
 .../SimulatorFullyImplicitBlackoilEbos.hpp       | 16 ++++++++++------
 2 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/opm/autodiff/SimulatorBase_impl.hpp b/opm/autodiff/SimulatorBase_impl.hpp
index cf80351c2..3e40f2db3 100644
--- a/opm/autodiff/SimulatorBase_impl.hpp
+++ b/opm/autodiff/SimulatorBase_impl.hpp
@@ -103,7 +103,12 @@ namespace Opm
         Opm::time::StopWatch total_timer;
         total_timer.start();
         std::string tstep_filename = output_writer_.outputDirectory() + "/step_timing.txt";
-        std::ofstream tstep_os(tstep_filename.c_str());
+        std::ofstream tstep_os;
+
+        if ( output_writer_.output() ) {
+            if ( output_writer_.isIORank() )
+                tstep_os.open(tstep_filename.c_str());
+        }
 
         const auto& schedule = eclipse_state_->getSchedule();
 
@@ -307,11 +312,8 @@ namespace Opm
                 OpmLog::note(msg);
             }
 
-            if ( output_writer_.output() ) {
-                if ( output_writer_.isIORank() )
-                {
+            if ( tstep_os.is_open() ) {
                     stepReport.reportParam(tstep_os);
-                }
             }
 
             // Increment timer, remember well state.
diff --git a/opm/autodiff/SimulatorFullyImplicitBlackoilEbos.hpp b/opm/autodiff/SimulatorFullyImplicitBlackoilEbos.hpp
index e7e839db6..35ab8ba54 100644
--- a/opm/autodiff/SimulatorFullyImplicitBlackoilEbos.hpp
+++ b/opm/autodiff/SimulatorFullyImplicitBlackoilEbos.hpp
@@ -163,7 +163,14 @@ public:
         Opm::time::StopWatch total_timer;
         total_timer.start();
         std::string tstep_filename = output_writer_.outputDirectory() + "/step_timing.txt";
-        std::ofstream tstep_os(tstep_filename.c_str());
+        std::ofstream tstep_os;
+
+        if ( output_writer_.output() ) {
+            if ( output_writer_.isIORank() )
+            {
+                tstep_os.open(tstep_filename.c_str());
+            }
+        }
 
         const auto& schedule = eclState().getSchedule();
 
@@ -351,11 +358,8 @@ public:
             // update timing.
             report.solver_time += solver_timer.secsSinceStart();
 
-            if ( output_writer_.output() ) {
-                if ( output_writer_.isIORank() )
-                {
-                    stepReport.reportParam(tstep_os);
-                }
+            if ( tstep_os.is_open() ) {
+                stepReport.reportParam(tstep_os);
             }
 
             // Increment timer, remember well state.