Updates after Conor's review

Framework/API/src/Algorithm.cpp

@@ -723,7 +723,7 @@ bool Algorithm::executeInternal() {
   }
 
   catch (...) {
-    // Execution
+    // Execution failed with an unknown exception object
     setResultState(ResultState::Failed);
     m_runningAsync = false;
 

Framework/API/src/AlgorithmProxy.cpp

@@ -109,7 +109,7 @@ bool AlgorithmProxy::executeAsyncImpl(const Poco::Void &dummy) {
     throw;
   }
   stopped();
-  return (resultState() == ResultState::Success);
+  return isExecuted();
 }
 
 /// Gets the current execution state