run_loop 1.2.0.pre1 → 1.2.0.pre2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 15eee499a372a51db5f46a5f922da9d776075fd6
-  data.tar.gz: c11c9303daa2a189a3e83d0f20565d89aaac8836
+  metadata.gz: dfc436b93536dc5fc25891e526a80a1b524dd94e
+  data.tar.gz: 119009225a74e03e009710319db28773d3d89e0c
 SHA512:
-  metadata.gz: 90aba28289c9d7ffaf2eb8724b5c25f1feaa068c7a1c8989ae7deec311ba71362e2675707e93ce8d031583e17e161b76026cc5ac611c66614bd15d4048eb66fa
-  data.tar.gz: 3d6913e3023a1a09c9e4a1a808a918e4b1ceda4a9a0b1ace9a693944254e54460ed7f47167f6d443213a873aba8a17e9021509fdec0910f14a29d95690e49e40
+  metadata.gz: 219265bba39959c8949ca278aa8e68d07d206f19cbbf6ffd660404de790ab3610e2bee7dde5f7454df9befa88b9ef40515b5132688ddf8978a2f7d360e5c8044
+  data.tar.gz: 1f02e149ce98e667c35194ae31cb8da6cc3f3f9621efacacf5a49385144395592b27e5387fd12c522a9325472c5579f231d7a9304f884f2607c1c9c08e1446c1

data/lib/run_loop/core.rb

@@ -11,6 +11,9 @@ module RunLoop
   class TimeoutError < RuntimeError
   end
 
+  class WriteFailedError < RuntimeError
+  end
+
   module Core
 
     START_DELIMITER = "OUTPUT_JSON:\n"
@@ -488,23 +491,47 @@ module RunLoop
       RUBY_PLATFORM == 'java'
     end
 
-    def self.write_request(run_loop, cmd)
+    def self.write_request(run_loop, cmd, logger=nil)
       repl_path = run_loop[:repl_path]
       index = run_loop[:index]
       cmd_str = "#{index}:#{escape_host_command(cmd)}"
-      log(cmd_str) if ENV['DEBUG'] == '1'
-      File.open(repl_path, 'w') { |f| f.puts(cmd_str) }
+      should_log = (ENV['DEBUG'] == '1')
+      RunLoop.log_info(logger, cmd_str) if should_log
+      write_succeeded = false
+      2.times do |i|
+        RunLoop.log_info(logger, "Trying write of command #{cmd_str} at index #{index}") if should_log
+        File.open(repl_path, 'w') { |f| f.puts(cmd_str) }
+        write_succeeded = validate_index_written(run_loop, index, logger)
+        break if write_succeeded
+      end
+      unless write_succeeded
+        RunLoop.log_info(logger, 'Failing...Raising RunLoop::WriteFailedError') if should_log
+        raise RunLoop::WriteFailedError.new("Trying write of command #{cmd_str} at index #{index}")
+      end
       run_loop[:index] = index + 1
 
       index
     end
 
+    def self.validate_index_written(run_loop, index, logger)
+      begin
+        Timeout::timeout(10, TimeoutError) do
+          Core.read_response(run_loop, index, 10, 'last_index')
+        end
+        RunLoop.log_info(logger, "validate index written for index #{index} ok")
+        return true
+      rescue TimeoutError => _
+        RunLoop.log_info(logger, "validate index written for index #{index} failed. Retrying.")
+        return false
+      end
+    end
+
     def self.escape_host_command(cmd)
       backquote = "\\"
       cmd.gsub(backquote,backquote*4)
     end
 
-    def self.read_response(run_loop, expected_index, empty_file_timeout=10)
+    def self.read_response(run_loop, expected_index, empty_file_timeout=10, search_for_property='index')
 
       log_file = run_loop[:log_file]
       initial_offset = run_loop[:initial_offset] || 0
@@ -566,7 +593,7 @@ module RunLoop
           if ENV['DEBUG_READ']=='1'
             p parsed_result
           end
-          json_index_if_present = parsed_result['index']
+          json_index_if_present = parsed_result[search_for_property]
           if json_index_if_present && json_index_if_present == expected_index
             result = parsed_result
             break
@@ -792,16 +819,12 @@ module RunLoop
     expected_index = run_loop[:index]
     result = nil
     begin
-      expected_index = Core.write_request(run_loop, cmd)
-    rescue Errno::EINTR => intr_error
+      expected_index = Core.write_request(run_loop, cmd, logger)
+    rescue RunLoop::WriteFailedError, Errno::EINTR => write_error
       # Attempt recover from interrupt by attempting to read result (assuming write went OK)
       # or retry if attempted read result fails
       run_loop[:index] = expected_index # restore expected index in case it changed
-      log_info(logger, "Core.write_request was interrupted: #{intr_error}. Attempting recovery...")
-      sleep(1) # Arbitrary wait in hope that the system condition causing the interrupt passes
-      if intr_error && intr_error.backtrace
-        log_info(logger, "backtrace: #{intr_error.backtrace.join("\n")}")
-      end
+      log_info(logger, "Core.write_request failed: #{write_error}. Attempting recovery...")
       log_info(logger, "Attempting read in case the request was received... Please wait (#{interrupt_retry_timeout})...")
       begin
         Timeout::timeout(interrupt_retry_timeout, TimeoutError) do
@@ -813,7 +836,7 @@ module RunLoop
         return result
       rescue TimeoutError => _
         log_info(logger, "Read did not result in a response for index #{expected_index}... Retrying send_command...")
-        return send_command(run_loop, cmd, options, num_retries+1, intr_error)
+        return send_command(run_loop, cmd, options, num_retries+1, write_error)
       end
     end
 

data/lib/run_loop/version.rb

@@ -1,5 +1,5 @@
 module RunLoop
-  VERSION = '1.2.0.pre1'
+  VERSION = '1.2.0.pre2'
 
   # A model of a software release version that can be used to compare two versions.
   #

data/scripts/run_loop_host.js

@@ -302,7 +302,7 @@ while (true) {
                 _actualIndex = parseInt(_input.substring(0, _index), 10);
                 if (!isNaN(_actualIndex) && _actualIndex >= _expectedIndex) {
                     _exp = _input.substring(_index + 1, _input.length);
-                    UIALogger.logMessage("Execute command: "+_exp);
+                    Log.output(_actualIndex);
                     _result = eval(_exp);
                 }
                 else {//likely old command is lingering...

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: run_loop
 version: !ruby/object:Gem::Version
-  version: 1.2.0.pre1
+  version: 1.2.0.pre2
 platform: ruby
 authors:
 - Karl Krukow
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-07 00:00:00.000000000 Z
+date: 2014-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor