QPR-12433 clean up at the beginning of run

Author: Roland Lichters

OREAnalytics/orea/app/oreapp.cpp

@@ -354,6 +354,7 @@ void OREApp::initFromParams() {
     CONSOLE("OK");
 
     Settings::instance().evaluationDate() = inputs_->asof();
+    LOG("initFromParameters done, requested analytics:" << to_string(inputs_->analytics()));
 } 
 
 void OREApp::initFromInputs() {
@@ -366,6 +367,7 @@ void OREApp::initFromInputs() {
 
     outputPath_ = inputs_->resultsPath().string();
     setupLog(outputPath_, logFile_, logMask_, logRootPath_);
+    LOG("initFromInputs done, requested analytics:" << to_string(inputs_->analytics()));
 }
 
 OREApp::~OREApp() {
@@ -375,24 +377,27 @@ OREApp::~OREApp() {
 
 void OREApp::run() {
 
-    // only one thread at a time should call run
+    // Only one thread at a time should call run
     static std::mutex _s_mutex;
     std::lock_guard<std::mutex> lock(_s_mutex);
 
-    // clean up after finishing the run
-    CleanUpThreadLocalSingletons cleanupThreadLocalSingletons;
-    CleanUpThreadGlobalSingletons cleanupThreadGloablSingletons;
-    CleanUpLogSingleton cleanupLogSingleton(true, true);
+    // Clean start, but leave Singletons intact after run is completed
+    {
+      CleanUpThreadLocalSingletons cleanupThreadLocalSingletons;
+      CleanUpThreadGlobalSingletons cleanupThreadGloablSingletons;
+      CleanUpLogSingleton cleanupLogSingleton(true, true);
+    }
 
-    if (inputs_ == nullptr)
-        initFromParams();
-    else if (params_ == nullptr)
+    // Use inputs when available, otherwise try params
+    if (inputs_ != nullptr)
         initFromInputs();
+    else if (params_ != nullptr)
+        initFromParams();
     else {
         ALOG("both inputs are empty");
-        return;      
+	return;
     }
-    
+
     runTimer_.start();
 
     try {