legion-data 1.10.2 → 1.10.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 212e01a164b58ab2a8a4273497e98acf0d99f174f4a66a77c7c707899685f2ed
-  data.tar.gz: e3302347dd4f8d3ce634fc88110a645ad6986aa5d8b531d5a9187fe894872ae0
+  metadata.gz: 2489246ac60704ae24678b24c75ab7d2e8f4153d332f3ad1d61f062a4e5c1c9c
+  data.tar.gz: 03e7b2217887a809525bdbf10337af382b1059638ce04c65c3c6bca31f517cc2
 SHA512:
-  metadata.gz: 6399d19fe35c9b3e975522d6d2b4654d0bc56240e508a38bcf55c4cb064fbb1214f7e2c5de7f3c4828a79f8543044da476811808d574b020609ab20adcc8d778
-  data.tar.gz: 87e1b1e58c66e464cbb1ccf41ad2a1917531edff611dcef0fc0f6b20f7dfe9331f854e684c00ace11557c714cd77e6131cd2a6b07c837f60d670c0d9fdf9b704
+  metadata.gz: 5bd7a8abc3a69bcf9ad7c7da89a3937057af7e10516b2070ea60eb0d893b9f09c83570b3535af5949ce0bf3138a7b4e6cd5302eac0bd0b09b259da84e432a9dd
+  data.tar.gz: b934c3058f683f0905a84590f41eb0da21db6ba8c9d221201e5a444c47e486c3cec1dedc14f058ebff7ea8c0637fe67d9bbbae66b108d211529909db3ef4e968

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Legion::Data Changelog
 
+## [1.10.3] - 2026-06-10
+
+### Changed
+- `connection_validation` now defaults to `false` — the Sequel connection_validator extension issues a `SELECT NULL` on every pool checkout/checkin and before real queries, which measurably degrades throughput; query-time error handling already recovers stale/dead connections. Set `connection_validation: true` in settings to opt back in (timeout semantics unchanged: `-1` validates every checkout)
+- Pool timeout assignments read `connection_validation_timeout` / `connection_expiration_timeout` directly from settings — removed inline `|| 600` / `|| 14_400` shadow defaults that disagreed with the documented settings defaults
+
 ## [1.10.2] - 2026-06-02
 
 ### Fixed

data/lib/legion/data/connection.rb

@@ -581,12 +581,12 @@ module Legion
 
           if data[:connection_validation] != false
             @sequel.extension(:connection_validator)
-            @sequel.pool.connection_validation_timeout = data[:connection_validation_timeout] || 600
+            @sequel.pool.connection_validation_timeout = data[:connection_validation_timeout]
           end
 
           if data[:connection_expiration] != false
             @sequel.extension(:connection_expiration)
-            @sequel.pool.connection_expiration_timeout = data[:connection_expiration_timeout] || 14_400
+            @sequel.pool.connection_expiration_timeout = data[:connection_expiration_timeout]
           end
         rescue StandardError => e
           handle_exception(e, level: :warn, handled: true, operation: :configure_extensions, adapter: adapter)

data/lib/legion/data/migrations/134_add_route_attempt_columns.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+Sequel.migration do
+  up do
+    alter_table(:llm_route_attempts) do
+      add_column :operation, String, size: 64, null: true
+      add_column :dispatch_path, String, size: 32, null: true
+      add_column :idempotency_key, String, size: 128, null: true
+      add_index :operation, name: :idx_route_attempts_operation
+      add_index :idempotency_key, name: :idx_route_attempts_idempotency_key
+    end
+  end
+
+  down do
+    alter_table(:llm_route_attempts) do
+      drop_index :operation, name: :idx_route_attempts_operation
+      drop_index :idempotency_key, name: :idx_route_attempts_idempotency_key
+      drop_column :operation
+      drop_column :dispatch_path
+      drop_column :idempotency_key
+    end
+  end
+end

data/lib/legion/data/settings.rb

@@ -48,8 +48,12 @@ module Legion
           sql_log_level:                 'debug',
 
           # Connection health (network adapters only, ignored for sqlite)
-          # -1 means validate on every checkout, catching stale connections from VPN/sleep/network changes immediately
-          connection_validation:         true,
+          # Validation is disabled by default: the connection_validator extension issues a
+          # SELECT NULL on every checkout/checkin and before real queries, which kills
+          # throughput. Connection errors are already rescued and reconnected at query time.
+          # When enabled, connection_validation_timeout: -1 validates on every checkout
+          # (catches stale connections from VPN/sleep/network changes immediately).
+          connection_validation:         false,
           connection_validation_timeout: -1,
           connection_expiration:         true,
           connection_expiration_timeout: 14_400,

data/lib/legion/data/version.rb

@@ -2,6 +2,6 @@
 
 module Legion
   module Data
-    VERSION = '1.10.2'
+    VERSION = '1.10.3'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: legion-data
 version: !ruby/object:Gem::Version
-  version: 1.10.2
+  version: 1.10.3
 platform: ruby
 authors:
 - Esity
@@ -280,6 +280,7 @@ files:
 - lib/legion/data/migrations/131_add_llm_tool_calls_schema_version.rb
 - lib/legion/data/migrations/132_drop_schema_version_from_llm_tool_calls.rb
 - lib/legion/data/migrations/133_allow_null_context_tokens.rb
+- lib/legion/data/migrations/134_add_route_attempt_columns.rb
 - lib/legion/data/model.rb
 - lib/legion/data/models/apollo/access_log.rb
 - lib/legion/data/models/apollo/entries.rb