clickhouse-native 0.4.1-aarch64-linux-gnu → 0.6.0-aarch64-linux-gnu

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 544deb2c372d0d8ddf4162e5099162960973cadbaeded92bb781486326330c70
-  data.tar.gz: cb55a90c3a90e536bf0f50d2f724d90e550868bfed19645822474d13d766264a
+  metadata.gz: f32459e08170ad3d746a116a2d18e20248ad1c02111a444bbd73fe985854548e
+  data.tar.gz: 3ce9cee80d2b7730b51433c2ffb3b1b3ab8516e4b490d83388dc2d4453efad3d
 SHA512:
-  metadata.gz: 8f22e31f5d6437a5bb5610479c633f14be0cb95e0a97b4b1c683fda2d199c127c40705dfffa378f466086da6d68193b3b59248f8ea2a41dc7330ecbb9d4428ca
-  data.tar.gz: 7b44b92b3a551fbbbb19c6933ad231a57b779a62a29797db60e36b417eb78ba55233507d7422717677f4cd2f7c5f9459785be472a81f1eaa33215def1ea4a6b4
+  metadata.gz: 6022e3a874dd4277764748154965fc7f1461f5f10a2d46b752db886337d0d9c1e4cf277dc5b456059a3ec65f81174568bcc5fb268507e845807539ac03f46423
+  data.tar.gz: b06d6ac4bb900bc0ab67b88f821a75604e5d34998d88b743302e81f481806d0e3435c70c700efc63dbde5d56902f70e680c0cf083d73f048d05f2a8df1bbe9a3

data/lib/clickhouse_native/client.rb

@@ -16,6 +16,10 @@ module ClickhouseNative
     # columns in DDL order (for Array<Array>).
     # types may be supplied to skip the DESCRIBE lookup.
     #
+    # The schema lookup is memoized per `(table, db_name)` on the client, so
+    # repeated inserts into the same table don't re-run DESCRIBE. Call
+    # `clear_schema_cache` after `ALTER TABLE` to invalidate.
+    #
     # Hash keys not present in the schema raise ArgumentError — if you need
     # to insert a subset, pass `columns:` explicitly.
     def insert(table, rows, columns: nil, db_name: nil, types: nil)
@@ -33,6 +37,14 @@ module ClickhouseNative
       insert_block(fq, col_pairs, row_arrays)
     end
 
+    # Drop the memoized schema for a table (or all tables). Needed after
+    # DDL that changes column set or types, since insert-time schema
+    # lookups are cached per (table, db_name).
+    def clear_schema_cache(table = nil, db_name: nil)
+      return unless defined?(@schema_cache) && @schema_cache
+      table.nil? ? @schema_cache.clear : @schema_cache.delete([db_name, table.to_s])
+    end
+
     def inspect
       "#<#{self.class} #{host}:#{port}/#{database}>"
     end
@@ -47,7 +59,7 @@ module ClickhouseNative
     end
 
     def columns_from_schema(table, rows, columns, db_name, fqn)
-      schema = describe_table(table, db_name: db_name)
+      schema = cached_schema(table, db_name: db_name)
       type_by_name = schema.to_h { |c| [c[:name], c[:type]] }
       columns ||= rows.first.is_a?(Hash) ? rows.first.keys.map(&:to_s) : schema.map { |c| c[:name] }
       columns.map do |name|
@@ -58,6 +70,12 @@ module ClickhouseNative
       end
     end
 
+    def cached_schema(table, db_name:)
+      @schema_cache ||= {}
+      @schema_cache[[db_name, table.to_s]] ||=
+        describe_table(table, db_name: db_name).freeze
+    end
+
     def hash_rows_to_arrays(rows, col_pairs)
       lookup = col_pairs.map { |n, _| [n.to_sym, n] }
       rows.map { |h| lookup.map { |sym, str| h.fetch(sym) { h[str] } } }

data/lib/clickhouse_native/pool.rb

@@ -4,8 +4,6 @@ require "connection_pool"
 
 module ClickhouseNative
   class Pool
-    STALE_IVAR = :@clickhouse_native_settings_stale
-
     attr_reader :host, :port, :database
 
     def initialize(host:, port:, database: "default", user: "default", password: "",
@@ -23,19 +21,20 @@ module ClickhouseNative
       end
     end
 
-    # Yields a client with the pool's session settings applied. If the
-    # previous checkout raised (which, in this gem's C++ bindings, always
-    # triggers a ResetConnection that wipes the session), re-apply the SET
-    # before yielding. Exceptions re-raise after marking the client stale.
+    # On exception, discard the client rather than reuse it: an error
+    # path leaves the socket in an unknown state. The C++ binding issues
+    # ResetConnection, but a subsequent send can still surface buffered
+    # protocol errors from the prior aborted operation — those get
+    # attributed to whatever SQL we tried next (e.g. the SET reapplying
+    # session settings), producing misleading log lines and re-raises in
+    # unrelated code. A fresh socket + handshake is cheap relative to
+    # debugging that.
     def with
       @pool.with do |client|
-        reapply_settings_if_stale(client)
-        begin
-          yield client
-        rescue
-          client.instance_variable_set(STALE_IVAR, true)
-          raise
-        end
+        yield client
+      rescue
+        @pool.discard_current_connection(&:close)
+        raise
       end
     end
 
@@ -73,13 +72,6 @@ module ClickhouseNative
 
     private
 
-    def reapply_settings_if_stale(client)
-      return unless @set_sql
-      return unless client.instance_variable_get(STALE_IVAR)
-      client.execute(@set_sql)
-      client.instance_variable_set(STALE_IVAR, false)
-    end
-
     # Render a `SET key1 = val1, key2 = val2` statement once at pool setup
     # so every checked-out connection starts with the same session
     # settings. Matches how the HTTP driver injected global_params per

data/lib/clickhouse_native/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ClickhouseNative
-  VERSION = "0.4.1"
+  VERSION = "0.6.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: clickhouse-native
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.6.0
 platform: aarch64-linux-gnu
 authors:
 - Yuri Smirnov
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 2.5.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 2.5.4
 description: A high-performance Ruby client for ClickHouse using the native binary
   protocol via a C++ extension wrapping clickhouse-cpp.
 email: