elasticsearch_record 1.2.1 → 1.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32688db9e3220d919f454f7c4c9445b006d43299d66322f4586af9d426fdc907
-  data.tar.gz: 9169c9110e662d647f7eddeddae7fa3497a03ad36486d85c001e762af5c756df
+  metadata.gz: cdb71bb8689c67cd7a2ef1add3b7ad0ab6710cbb5833313cb3f474267c39e43e
+  data.tar.gz: 06ec37ac16f5adb110da8ffa08c9c44fc20993be3db12bf8033315cfa9611fc3
 SHA512:
-  metadata.gz: 9221a04be02abcb7d8f36da24f3b7e2d061b778233b8190bb14b906bf54b2a8d881a870f9c0b1b65d90b3ff2418fe8ee21d828b35dceac85b2f8bc766677c73c
-  data.tar.gz: a1b4e64cf4a8c33d8ce3aaeb89fedd5b7847a648a1dcfccc2b15ecfba0f99d0fbf9a0c94237edcd6ef0feb8d4ef40889ab00c2b8a6bc50773b4982cd5e326149
+  metadata.gz: 929f5df6456203e7bce466d62836805bb9a75ff5a89856c75cd1ba187a95e05f51aa3ad676c3058d7ad95d1295db1422571964240182000bef59a8cdb95181bc
+  data.tar.gz: 7f785864f92bc6f3243ea081236327ff7c600833cdf74a11d59239f31551d928dc7b8bb1b4952ec38aadaa8d0174ac2a7ceeebae100a247c3534ef4c1609e484

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    elasticsearch_record (1.2.1)
+    elasticsearch_record (1.2.3)
       activerecord (~> 7.0.0)
       elasticsearch (~> 8.4)
 

data/docs/CHANGELOG.md

@@ -1,5 +1,14 @@
 # ElasticsearchRecord - CHANGELOG
 
+## [1.2.3] - 2022-12-12
+* [fix] `change_table` 'if_exists: true' returns at the wrong state
+
+## [1.2.2] - 2022-12-12
+* [add] `:if_exists` option for `change_table`
+* [fix] executing `_compute_table_name` irregular on some schema methods and some not - not only executes on `create_table` and `change_table`
+* [ref] private `_compute_table_name` method to public `compute_table_name`
+* [ref] drop `_compute_table_name` on methods: open_table, close_table, truncate_table, drop_table, block_table, unblock_table & clone_table
+
 ## [1.2.1] - 2022-12-12
 * [add] `ActiveRecord::ConnectionAdapters::Elasticsearch::SchemaStatements#access_id_fielddata?` which checks the clusters setting 'indices.id_field_data.enabled' to determinate if a general sorting on the +_id+-field is possible or not.
 * [add] `ElasticsearchRecord::Relation#ordered_relation` which overwrites the original method to check against the `#access_id_fielddata?` method

data/lib/active_record/connection_adapters/elasticsearch/table_statements.rb

@@ -38,9 +38,6 @@ module ActiveRecord
           # @param [String] table_name
           # @return [Boolean] acknowledged status
           def open_table(table_name)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             schema_cache.clear_data_source_cache!(table_name)
             api(:indices, :open, { index: table_name }, 'OPEN TABLE').dig('acknowledged')
           end
@@ -59,9 +56,6 @@ module ActiveRecord
           # @param [String] table_name
           # @return [Boolean] acknowledged status
           def close_table(table_name)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             schema_cache.clear_data_source_cache!(table_name)
             api(:indices, :close, { index: table_name }, 'CLOSE TABLE').dig('acknowledged')
           end
@@ -84,9 +78,6 @@ module ActiveRecord
           # @param [String] table_name
           # @return [Boolean] acknowledged status
           def truncate_table(table_name)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             # force: automatically drops an existing index
             create_table(table_name, force: true, **table_schema(table_name))
           end
@@ -111,9 +102,6 @@ module ActiveRecord
           # @param [Boolean] if_exists
           # @return [Array] acknowledged status
           def drop_table(table_name, if_exists: false, **)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             schema_cache.clear_data_source_cache!(table_name)
             api(:indices, :delete, { index: table_name, ignore: (if_exists ? 404 : nil) }, 'DROP TABLE').dig('acknowledged')
           end
@@ -123,9 +111,6 @@ module ActiveRecord
           # @param [Symbol] block_name The block to add (one of :read, :write, :read_only or :metadata)
           # @return [Boolean] acknowledged status
           def block_table(table_name, block_name = :write)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             api(:indices, :add_block, { index: table_name, block: block_name }, "BLOCK #{block_name.to_s.upcase} TABLE").dig('acknowledged')
           end
 
@@ -135,9 +120,6 @@ module ActiveRecord
           # @param [Symbol] block_name The block to add (one of :read, :write, :read_only or :metadata)
           # @return [Boolean] acknowledged status
           def unblock_table(table_name, block_name = nil)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-
             if block_name.nil?
               change_table(table_name) do |t|
                 t.change_setting('index.blocks.read', false)
@@ -157,10 +139,6 @@ module ActiveRecord
           # @param [Hash] options
           # @param [Proc] block
           def clone_table(table_name, target_name, **options, &block)
-            # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
-            target_name = _compute_table_name(target_name)
-
             # create new definition
             definition = clone_table_definition(table_name, target_name, **extract_table_options!(options), &block)
 
@@ -191,7 +169,7 @@ module ActiveRecord
           # @return [Boolean] acknowledged status
           def create_table(table_name, force: false, copy_from: nil, if_not_exists: false, **options)
             # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
+            table_name = compute_table_name(table_name)
 
             return if if_not_exists && table_exists?(table_name)
 
@@ -226,9 +204,11 @@ module ActiveRecord
           #     t.mapping :name, :string
           #     # Other column alterations here
           #   end
-          def change_table(table_name, **options)
+          def change_table(table_name, if_exists: false, **options)
             # IMPORTANT: compute will add possible configured prefix & suffix
-            table_name = _compute_table_name(table_name)
+            table_name = compute_table_name(table_name)
+
+            return if if_exists && !table_exists?(table_name)
 
             definition = update_table_definition(table_name, self, **options)
 
@@ -303,17 +283,22 @@ module ActiveRecord
             _exec_change_table_with(:delete_alias, table_name, name, **options, &block)
           end
 
-          private
-
-          def _compute_table_name(table_name)
+          # computes a provided +table_name+ with optionally configured +table_name_prefix+ & +table_name_suffix+.
+          # @param [String] table_name
+          # @return [String]
+          def compute_table_name(table_name)
+            table_name = table_name.to_s
+            
             # HINT: +"" creates a new +unfrozen+ string!
             str = +""
-            str << table_name_prefix unless table_name.to_s.start_with?(table_name_prefix)
+            str << table_name_prefix unless table_name.start_with?(table_name_prefix)
             str << table_name
-            str << table_name_suffix unless table_name.to_s.end_with?(table_name_suffix)
+            str << table_name_suffix unless table_name.end_with?(table_name_suffix)
             str
           end
 
+          private
+
           def _exec_change_table_with(method, table_name, *args, **kwargs, &block)
             change_table(table_name) do |t|
               t.send(method, *args, **kwargs, &block)

data/lib/elasticsearch_record/gem_version.rb

@@ -9,7 +9,7 @@ module ElasticsearchRecord
   module VERSION
     MAJOR = 1
     MINOR = 2
-    TINY  = 1
+    TINY  = 3
     PRE   = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: elasticsearch_record
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.3
 platform: ruby
 authors:
 - Tobias Gonsior