Fix always-break-connection issue in executor service

Author: Alexander Avramenko

api/include/apihandler.hpp

@@ -470,7 +470,7 @@ public slots:
                     });
                 }
 
-                static const int kReconnectTime = 5;
+                static const int kReconnectTime = 2;
                 std::this_thread::sleep_for(std::chrono::seconds(kReconnectTime));
 
                 if (!isConnect()) {
@@ -523,10 +523,6 @@ public slots:
 
     bool connect() {
         try {
-            if (executorTransport_->isOpen()) {
-                executorTransport_->close();
-            }
-
             executorTransport_->open();
         }
         catch (...) {

api/src/apihandler.cpp

@@ -2220,9 +2220,10 @@ namespace executor {
 
     void Executor::executeByteCodeMultiple(ExecuteByteCodeMultipleResult& _return, const ::general::Address& initiatorAddress, const SmartContractBinary& invokedContract,
         const std::string& method, const std::vector<std::vector<::general::Variant>>& params, const int64_t executionTime, cs::Sequence sequence) {
-        if (!connect()) {
+        if (!isConnect()) {
             _return.status.code = 1;
             _return.status.message = "No executor connection!";
+            notifyError();
             return;
         }
         const auto access_id = generateAccessId(sequence);
@@ -2584,7 +2585,8 @@ namespace executor {
         const SmartContractBinary& smartContractBinary, std::vector<MethodHeader>& methodHeader, bool isGetter, cs::Sequence explicit_sequence) {
         constexpr uint64_t EXECUTION_TIME = Consensus::T_smart_contract;
         OriginExecuteResult originExecuteRes{};
-        if (!connect()) {
+        if (!isConnect()) {
+            notifyError();
             return std::nullopt;
         }