polyrun 2.2.4 → 2.2.6

data/lib/polyrun/spec_quality/report.rb

@@ -158,13 +158,7 @@ module Polyrun
           violations << "zero_hit_examples=#{analysis[:zero_hit].size} max=#{max_zero}"
         end
 
-        min_unique = cfg["minimum_unique_lines_per_example"]
-        if min_unique
-          bad = analysis[:zero_hit].keys
-          if bad.size.positive?
-            violations << "examples_below_minimum_unique_lines=#{bad.size} minimum=#{min_unique}"
-          end
-        end
+        violations.concat(minimum_unique_line_violations(merged, cfg))
 
         max_hot = cfg["max_hot_line_overlap"]
         if max_hot
@@ -175,6 +169,21 @@ module Polyrun
         violations
       end
 
+      def minimum_unique_line_violations(merged, cfg)
+        violations = []
+        min_unique = cfg["minimum_unique_lines_per_example"]
+        if min_unique
+          threshold = min_unique.to_i
+          examples = merged["examples"] || {}
+          bad = examples.select { |_loc, row| row["unique_lines"].to_i < threshold }.keys
+          if bad.size.positive?
+            violations << "examples_below_minimum_unique_lines=#{bad.size} minimum=#{min_unique}"
+          end
+        end
+
+        violations
+      end
+
       def emit_warnings!(merged, cfg = {})
         analyze(merged, cfg).fetch(:line_churn, {}).each do |loc, row|
           Polyrun::Log.warn "polyrun spec-quality line_churn: #{loc} churn=#{row["line_churn"]}"

data/lib/polyrun/templates/rails_prepare.polyrun.yml

@@ -22,7 +22,9 @@ prepare:
 #   merge_spec_quality: true
 #   merge_spec_quality_output: coverage/polyrun-spec-quality.json
 
-# Uncomment and edit when using Postgres template + per-shard DB names from polyrun env:
+# Uncomment and edit when using Postgres template + per-shard DB names from polyrun env.
+# CloneShards / polyrun start auto provision is PostgreSQL-only. For mysql2 (or other adapters),
+# keep template_db + shard_db_pattern for URL shaping, set start.databases: false, and provision shards in prepare.
 # databases:
 #   template_db: my_app_template
 #   shard_db_pattern: "my_app_test_%{shard}"

data/lib/polyrun/version.rb

@@ -1,3 +1,3 @@
 module Polyrun
-  VERSION = "2.2.4"
+  VERSION = "2.2.6"
 end

data/lib/polyrun/worker_output.rb

@@ -55,20 +55,37 @@ module Polyrun
       out_read, out_write = IO.pipe
       err_read, err_write = IO.pipe
 
-      pid =
-        if hook_cfg.worker_hooks? && !Polyrun::Hooks.disabled?
-          Process.spawn(
-            child_env,
-            "sh",
-            "-c",
-            hook_cfg.build_worker_shell_script(cmd, paths),
-            out: out_write,
-            err: err_write
-          )
-        else
-          # Array(cmd) so a String command is one argv element (not character-splatted).
-          Process.spawn(child_env, *Array(cmd), *Array(paths), out: out_write, err: err_write)
+      begin
+        pid =
+          if hook_cfg.worker_hooks? && !Polyrun::Hooks.disabled?
+            Process.spawn(
+              child_env,
+              "sh",
+              "-c",
+              hook_cfg.build_worker_shell_script(cmd, paths),
+              out: out_write,
+              err: err_write,
+              pgroup: true
+            )
+          else
+            # Array(cmd) so a String command is one argv element (not character-splatted).
+            Process.spawn(
+              child_env,
+              *Array(cmd),
+              *Array(paths),
+              out: out_write,
+              err: err_write,
+              pgroup: true
+            )
+          end
+      rescue
+        [out_read, out_write, err_read, err_write].each do |io|
+          io.close unless io.closed?
+        rescue IOError
+          nil
         end
+        raise
+      end
 
       out_write.close
       err_write.close

data/lib/polyrun.rb

@@ -11,6 +11,7 @@ require_relative "polyrun/coverage/reporting"
 require_relative "polyrun/coverage/rails"
 require_relative "polyrun/partition/plan"
 require_relative "polyrun/partition/paths"
+require_relative "polyrun/partition/suite"
 require_relative "polyrun/partition/paths_build"
 require_relative "polyrun/queue/file_store"
 require_relative "polyrun/data/fixtures"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: polyrun
 version: !ruby/object:Gem::Version
-  version: 2.2.4
+  version: 2.2.6
 platform: ruby
 authors:
 - Andrei Makarov
@@ -313,6 +313,7 @@ files:
 - lib/polyrun/partition/plan_sharding.rb
 - lib/polyrun/partition/reports.rb
 - lib/polyrun/partition/stable_shuffle.rb
+- lib/polyrun/partition/suite.rb
 - lib/polyrun/partition/timing_diagnostics.rb
 - lib/polyrun/partition/timing_keys.rb
 - lib/polyrun/prepare/artifacts.rb
@@ -320,6 +321,7 @@ files:
 - lib/polyrun/process_stdio.rb
 - lib/polyrun/queue/duration.rb
 - lib/polyrun/queue/file_store.rb
+- lib/polyrun/queue/file_store_intent.rb
 - lib/polyrun/queue/file_store_pending.rb
 - lib/polyrun/queue/worker_loop.rb
 - lib/polyrun/quick.rb