elasticsearch-api 8.0.0.pre1 → 8.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99c344ac040a814342dad8d82f4f4b01fcb13fb8a98a1782bf381297efa53d99
-  data.tar.gz: 4b3778473c7384d433d9c9df4cf94b6e5f13b605c258e2f2781e4a1046272228
+  metadata.gz: ba0c493d151fc85f4eeb16668d642cfcebda58124d1c269d6ef26503b4206b13
+  data.tar.gz: b9b63b6f73bc091e7525e01ec48b2b1b0af77c5778fb3d5f604a88dae1931351
 SHA512:
-  metadata.gz: 79009142d3cdb47fbab8ade4b2d66949bc1ef81f33653f17c1db9eceb745e34440a87efca4b8253fae189c52172890f978a2c2a8d11b1013cc469bc6503f5f40
-  data.tar.gz: c30ef81f519d5d61acd0ffa37f34fbe267ac8acba5f4ff48e493a39f0724ef4b07116a4674e2cba8ae74febd55bfc168349e1a5e02336b67c778068c33b714bd
+  metadata.gz: 3456a52c9d8e19a49a19d0e51335d1b5a7c728c4f09c525c0faa79ddf567715cc7b01b1dd88d99f472c2e005272daf29c3ee56426d94598522300b2876d4f742
+  data.tar.gz: 961c778643af216e0a3865293dd8a17342d164944e717b081d9a22f1bae404c027fc6c13f0a343a7ebdd72e9e93dc45e53a201bbe7d4c3f0e717b4ffc1e0de38

data/api-spec-testing/rspec_matchers.rb

@@ -245,7 +245,7 @@ RSpec::Matchers.define :match_response do |pairs, test|
       # When the value in the yaml test is a big number, the format is
       # different from what Ruby uses, so we transform  X.XXXXEXX to X.XXXXXe+XX
       # to be able to compare the values
-      actual_value.to_s == expected.gsub('E', 'e+')
+      actual_value.to_s == expected.gsub('E', 'e+') || actual_value == expected
     elsif expected == '' && actual_value != ''
       actual_value == response
     else

data/api-spec-testing/test_file/action.rb

@@ -54,59 +54,82 @@ module Elasticsearch
       # @since 6.2.0
       def execute(client, test = nil)
         @definition.each.inject(client) do |client, (method_chain, args)|
-          chain = method_chain.split('.')
-
-          # If we have a method nested in a namespace, client becomes the
-          # client/namespace. Eg for `indices.resolve_index`, `client =
-          # client.indices` and then we call `resolve_index` on `client`.
-          if chain.size > 1
-            client = chain[0...-1].inject(client) do |_client, _method|
-              _client.send(_method)
+          if method_chain.match?('_internal')
+            es_version = test.cached_values['es_version'] unless test.nil?
+            perform_internal(method_chain, args, client, es_version)
+          else
+            chain = method_chain.split('.')
+            # If we have a method nested in a namespace, client becomes the
+            # client/namespace. Eg for `indices.resolve_index`, `client =
+            # client.indices` and then we call `resolve_index` on `client`.
+            if chain.size > 1
+              client = chain[0...-1].inject(client) do |shadow_client, method|
+                shadow_client.send(method)
+              end
             end
-          end
 
-          _method = chain[-1]
-          case _method
-          when 'headers'
-            headers = prepare_arguments(args, test)
-            # TODO: Remove Authorization headers while x_pack_rest_user is fixed
-            if headers[:Authorization] == 'Basic eF9wYWNrX3Jlc3RfdXNlcjp4LXBhY2stdGVzdC1wYXNzd29yZA=='
-              headers.delete(:Authorization)
-            end
-            host = client.transport.instance_variable_get('@hosts')
-            transport_options = client.transport.instance_variable_get('@options')&.dig(:transport_options) || {}
-            if ENV['QUIET'] == 'true'
-              # todo: create a method on Elasticsearch::Client that can clone the client with new options
-              Elasticsearch::Client.new(
-                host: host,
-                transport_options: transport_options.merge(headers: headers)
-              )
-            else
-              Elasticsearch::Client.new(
-                host: host,
-                tracer: Logger.new($stdout),
-                transport_options: transport_options.merge(headers: headers)
-              )
-            end
-          when 'catch', 'warnings', 'allowed_warnings', 'allowed_warnings_regex'
-            client
-          when 'put_trained_model_alias'
-            args.merge!('reassign' => true) unless args['reassign'] === false
-            @response = client.send(_method, prepare_arguments(args, test))
-            client
-          when 'create'
-            begin
+            _method = chain[-1]
+            case _method
+            when 'bulk'
+              arguments = prepare_arguments(args, test)
+              arguments[:body].map! do |item|
+                if item.is_a?(Hash)
+                  item
+                elsif item.is_a?(String)
+                  symbolize_keys(JSON.parse(item))
+                end
+              end if arguments[:body].is_a? Array
+              @response = client.send(_method, arguments)
+              client
+            when 'headers'
+              headers = prepare_arguments(args, test)
+              # TODO: Remove Authorization headers while x_pack_rest_user is fixed
+              if headers[:Authorization] == 'Basic eF9wYWNrX3Jlc3RfdXNlcjp4LXBhY2stdGVzdC1wYXNzd29yZA=='
+                headers.delete(:Authorization)
+              end
+              host = client.transport.instance_variable_get('@hosts')
+              transport_options = client.transport.instance_variable_get('@options')&.dig(:transport_options) || {}
+              if ENV['QUIET'] == 'true'
+                # todo: create a method on Elasticsearch::Client that can clone the client with new options
+                Elasticsearch::Client.new(
+                  host: host,
+                  transport_options: transport_options.merge(headers: headers)
+                )
+              else
+                Elasticsearch::Client.new(
+                  host: host,
+                  tracer: Logger.new($stdout),
+                  transport_options: transport_options.merge(headers: headers)
+                )
+              end
+            when 'catch', 'warnings', 'allowed_warnings', 'allowed_warnings_regex'
+              client
+            when 'put_trained_model_alias'
+              args.merge!('reassign' => true) unless args['reassign'] === false
               @response = client.send(_method, prepare_arguments(args, test))
-            rescue Elastic::Transport::Transport::Errors::BadRequest => e
-              raise e unless e.message.match 'resource_already_exists_exception'
-
-              client.delete(index: args['index'])
+              client
+            when 'create'
+              begin
+                @response = client.send(_method, prepare_arguments(args, test))
+              rescue Elastic::Transport::Transport::Errors::BadRequest => e
+                case e.message
+                when /resource_already_exists_exception/
+                  client.delete(index: args['index'])
+                when /failed to parse date field/
+                  body = args['body']
+                  time_series = body['settings']['index']['time_series']
+                  time_series.each { |k, v| time_series[k] = v.strftime("%FT%TZ") }
+                  args['body'] = body
+                else
+                  raise e
+                end
+                @response = client.send(_method, prepare_arguments(args, test))
+              end
+              client
+            else
               @response = client.send(_method, prepare_arguments(args, test))
+              client
             end
-            client
-          else
-            @response = client.send(_method, prepare_arguments(args, test))
-            client
           end
         end
       end
@@ -117,6 +140,38 @@ module Elasticsearch
 
       private
 
+      # Executes operations not implemented by elasticsearch-api, such as _internal
+      def perform_internal(method, args, client, es_version)
+        case method
+        when '_internal.update_desired_nodes'
+          http = 'PUT'
+          if (history_id = args.delete('history_id')).match?(/\s+/)
+            require 'erb'
+            history_id = ERB::Util.url_encode(history_id)
+          end
+          path = "/_internal/desired_nodes/#{history_id}/#{args.delete('version')}"
+          body = args.delete('body')
+          # Replace $es_version with actual value:
+          body['nodes'].map do |node|
+            node['node_version']&.gsub!('$es_version', es_version) if node['node_version'] && es_version
+          end if body['nodes']
+        when '_internal.delete_desired_nodes'
+          http = 'DELETE'
+          path = '/_internal/desired_nodes/'
+          body = args.delete('body')
+        when '_internal.get_desired_nodes'
+          http = 'GET'
+          path = '/_internal/desired_nodes/_latest'
+          body = nil
+        when '_internal.health'
+          http = 'GET'
+          path = '_internal/_health'
+          body = args.delete('body')
+        end
+        @response = Elasticsearch::API::Response.new(client.perform_request(http, path, args, body))
+        client
+      end
+
       def prepare_arguments(args, test)
         symbolize_keys(args).tap do |args|
           if test

data/api-spec-testing/test_file.rb

@@ -49,22 +49,28 @@ module Elasticsearch
         @client = client
         begin
           documents = YAML.load_stream(File.new(file_name))
+        rescue Psych::SyntaxError => e
+          raise e unless e.message.include?('found unexpected \':\'')
+
+          message = "Exception found when parsing YAML in #{file_name}: #{e.message}"
+          logger.error message
+          raise SkipTestsException, message
         rescue StandardError => e
           logger.error e
           logger.error "Filename : #{@name}"
         end
         @test_definitions = documents.reject { |doc| doc['setup'] || doc['teardown'] }
         @setup = documents.find { |doc| doc['setup'] }
-        skip_entire_test_file? if @setup
+        skip_entire_test_file?(file_name) if @setup
         @teardown = documents.find { |doc| doc['teardown'] }
         @features_to_skip = REST_API_YAML_SKIP_FEATURES + features_to_skip
       end
 
-      def skip_entire_test_file?
+      def skip_entire_test_file?(file_name)
         @skip = @setup['setup']&.select { |a| a['skip'] }
         return false if @skip.empty?
 
-        raise SkipTestsException if skip_version?(@client, @skip.first['skip'])
+        raise SkipTestsException, "Skipping #{file_name} due to 'skip all'." if skip_version?(@client, @skip.first['skip'])
       end
 
       def skip_version?(client, skip_definition)

data/api-spec-testing/wipe_cluster.rb

@@ -25,7 +25,8 @@ module Elasticsearch
         'ilm-history-ilm-policy', 'slm-history-ilm-policy', 'watch-history-ilm-policy',
         'ml-size-based-ilm-policy', 'logs', 'metrics', 'synthetics', '7-days-default',
         '30-days-default', '90-days-default', '180-days-default', '365-days-default',
-        '.fleet-actions-results-ilm-policy', '.deprecation-indexing-ilm-policy'
+        '.fleet-actions-results-ilm-policy', '.deprecation-indexing-ilm-policy',
+        'watch-history-ilm-policy-16', '.monitoring-8-ilm-policy'
       ].freeze
 
       PLATINUM_TEMPLATES = [

data/elasticsearch-api.gemspec

@@ -60,10 +60,17 @@ Gem::Specification.new do |s|
   # Gems for testing integrations
   s.add_development_dependency 'jsonify'
   s.add_development_dependency 'hashie'
+  # Temporary support for Ruby 2.6, since it's EOL March 2022:
+  if RUBY_VERSION < '2.7.0'
+    s.add_development_dependency 'jbuilder', '< 7.0.0'
+  else
+    s.add_development_dependency 'activesupport'
+    s.add_development_dependency 'jbuilder'
+  end
 
   s.add_development_dependency 'cane'
   s.add_development_dependency 'escape_utils' unless defined? JRUBY_VERSION
-  s.add_development_dependency 'jbuilder'
+
   s.add_development_dependency 'require-prof' unless defined?(JRUBY_VERSION) || defined?(Rubinius)
   s.add_development_dependency 'ruby-prof' unless defined?(JRUBY_VERSION) || defined?(Rubinius)
   s.add_development_dependency 'simplecov'

data/lib/elasticsearch/api/actions/async_search/delete.rb

@@ -24,17 +24,16 @@ module Elasticsearch
         # @option arguments [String] :id The async search ID
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/async-search.html
         #
         def delete(arguments = {})
           raise ArgumentError, "Required argument 'id' missing" unless arguments[:id]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _id = arguments.delete(:id)
 
           method = Elasticsearch::API::HTTP_DELETE

data/lib/elasticsearch/api/actions/async_search/get.rb

@@ -27,17 +27,16 @@ module Elasticsearch
         # @option arguments [Boolean] :typed_keys Specify whether aggregation and suggester names should be prefixed by their respective types in the response
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/async-search.html
         #
         def get(arguments = {})
           raise ArgumentError, "Required argument 'id' missing" unless arguments[:id]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _id = arguments.delete(:id)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/async_search/status.rb

@@ -24,17 +24,16 @@ module Elasticsearch
         # @option arguments [String] :id The async search ID
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/async-search.html
         #
         def status(arguments = {})
           raise ArgumentError, "Required argument 'id' missing" unless arguments[:id]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _id = arguments.delete(:id)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/async_search/submit.rb

@@ -66,14 +66,13 @@ module Elasticsearch
         # @option arguments [Hash] :headers Custom HTTP headers
         # @option arguments [Hash] :body The search definition using the Query DSL
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/async-search.html
         #
         def submit(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = arguments.delete(:body)
-
-          arguments = arguments.clone
+          body   = arguments.delete(:body)
 
           _index = arguments.delete(:index)
 

data/lib/elasticsearch/api/actions/autoscaling/delete_autoscaling_policy.rb

@@ -24,17 +24,16 @@ module Elasticsearch
         # @option arguments [String] :name the name of the autoscaling policy
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-delete-autoscaling-policy.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/autoscaling-delete-autoscaling-policy.html
         #
         def delete_autoscaling_policy(arguments = {})
           raise ArgumentError, "Required argument 'name' missing" unless arguments[:name]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _name = arguments.delete(:name)
 
           method = Elasticsearch::API::HTTP_DELETE

data/lib/elasticsearch/api/actions/autoscaling/get_autoscaling_capacity.rb

@@ -23,14 +23,13 @@ module Elasticsearch
         #
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-get-autoscaling-capacity.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/autoscaling-get-autoscaling-capacity.html
         #
         def get_autoscaling_capacity(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           method = Elasticsearch::API::HTTP_GET
           path   = "_autoscaling/capacity"

data/lib/elasticsearch/api/actions/autoscaling/get_autoscaling_policy.rb

@@ -24,17 +24,16 @@ module Elasticsearch
         # @option arguments [String] :name the name of the autoscaling policy
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-get-autoscaling-policy.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/autoscaling-get-autoscaling-policy.html
         #
         def get_autoscaling_policy(arguments = {})
           raise ArgumentError, "Required argument 'name' missing" unless arguments[:name]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _name = arguments.delete(:name)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/autoscaling/put_autoscaling_policy.rb

@@ -25,18 +25,17 @@ module Elasticsearch
         # @option arguments [Hash] :headers Custom HTTP headers
         # @option arguments [Hash] :body the specification of the autoscaling policy (*Required*)
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-put-autoscaling-policy.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/autoscaling-put-autoscaling-policy.html
         #
         def put_autoscaling_policy(arguments = {})
           raise ArgumentError, "Required argument 'body' missing" unless arguments[:body]
           raise ArgumentError, "Required argument 'name' missing" unless arguments[:name]
 
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = arguments.delete(:body)
 
-          arguments = arguments.clone
-
           _name = arguments.delete(:name)
 
           method = Elasticsearch::API::HTTP_PUT

data/lib/elasticsearch/api/actions/bulk.rb

@@ -21,11 +21,11 @@ module Elasticsearch
       # Allows to perform multiple index/update/delete operations in a single request.
       #
       # @option arguments [String] :index Default index for items which don't provide one
-      # @option arguments [String] :type Default document type for items which don't provide one
       # @option arguments [String] :wait_for_active_shards Sets the number of shard copies that must be active before proceeding with the bulk operation. Defaults to 1, meaning the primary shard only. Set to `all` for all shard copies, otherwise set to any non-negative value less than or equal to the total number of copies for the shard (number of replicas + 1)
       # @option arguments [String] :refresh If `true` then refresh the affected shards to make this operation visible to search, if `wait_for` then wait for a refresh to make this operation visible to search, if `false` (the default) then do nothing with refreshes. (options: true, false, wait_for)
       # @option arguments [String] :routing Specific routing value
       # @option arguments [Time] :timeout Explicit operation timeout
+      # @option arguments [String] :type Default document type for items which don't provide one
       # @option arguments [List] :_source True or false to return the _source field or not, or default list of fields to return, can be overridden on each sub-request
       # @option arguments [List] :_source_excludes Default list of fields to exclude from the returned _source field, can be overridden on each sub-request
       # @option arguments [List] :_source_includes Default list of fields to extract and return from the _source field, can be overridden on each sub-request
@@ -35,25 +35,20 @@ module Elasticsearch
       # @option arguments [String|Array] :body The operation definition and data (action-data pairs), separated by newlines. Array of Strings, Header/Data pairs,
       # or the conveniency "combined" format can be passed, refer to Elasticsearch::API::Utils.__bulkify documentation.
       #
-      # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/docs-bulk.html
+      # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/docs-bulk.html
       #
       def bulk(arguments = {})
         raise ArgumentError, "Required argument 'body' missing" unless arguments[:body]
 
+        arguments = arguments.clone
         headers = arguments.delete(:headers) || {}
 
-        body = arguments.delete(:body)
-
-        arguments = arguments.clone
+        body   = arguments.delete(:body)
 
         _index = arguments.delete(:index)
 
-        _type = arguments.delete(:type)
-
         method = Elasticsearch::API::HTTP_POST
-        path   = if _index && _type
-                   "#{Utils.__listify(_index)}/#{Utils.__listify(_type)}/_bulk"
-                 elsif _index
+        path   = if _index
                    "#{Utils.__listify(_index)}/_bulk"
                  else
                    "_bulk"

data/lib/elasticsearch/api/actions/cat/aliases.rb

@@ -31,15 +31,14 @@ module Elasticsearch
         # @option arguments [String] :expand_wildcards Whether to expand wildcard expression to concrete indices that are open, closed or both. (options: open, closed, hidden, none, all)
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-alias.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-alias.html
         #
         def aliases(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _name = arguments.delete(:name)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/cat/allocation.rb

@@ -32,15 +32,14 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-allocation.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-allocation.html
         #
         def allocation(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _node_id = arguments.delete(:node_id)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/cat/count.rb

@@ -29,14 +29,13 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-count.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-count.html
         #
         def count(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           _index = arguments.delete(:index)
 

data/lib/elasticsearch/api/actions/cat/fielddata.rb

@@ -30,15 +30,14 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-fielddata.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-fielddata.html
         #
         def fielddata(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _fields = arguments.delete(:fields)
 
           method = Elasticsearch::API::HTTP_GET

data/lib/elasticsearch/api/actions/cat/health.rb

@@ -30,14 +30,13 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-health.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-health.html
         #
         def health(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           method = Elasticsearch::API::HTTP_GET
           path   = "_cat/health"

data/lib/elasticsearch/api/actions/cat/help.rb

@@ -25,14 +25,13 @@ module Elasticsearch
         # @option arguments [List] :s Comma-separated list of column names or column aliases to sort by
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat.html
         #
         def help(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           method = Elasticsearch::API::HTTP_GET
           path   = "_cat"

data/lib/elasticsearch/api/actions/cat/indices.rb

@@ -36,14 +36,13 @@ module Elasticsearch
         # @option arguments [String] :expand_wildcards Whether to expand wildcard expression to concrete indices that are open, closed or both. (options: open, closed, hidden, none, all)
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-indices.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-indices.html
         #
         def indices(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           _index = arguments.delete(:index)
 

data/lib/elasticsearch/api/actions/cat/master.rb

@@ -30,14 +30,13 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see https://www.elastic.co/guide/en/elasticsearch/reference/master/cat-master.html
+        # @see https://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-master.html
         #
         def master(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
-          body = nil
-
-          arguments = arguments.clone
+          body   = nil
 
           method = Elasticsearch::API::HTTP_GET
           path   = "_cat/master"

data/lib/elasticsearch/api/actions/cat/ml_data_frame_analytics.rb

@@ -32,15 +32,14 @@ module Elasticsearch
         # @option arguments [Boolean] :v Verbose mode. Display column headers
         # @option arguments [Hash] :headers Custom HTTP headers
         #
-        # @see http://www.elastic.co/guide/en/elasticsearch/reference/current/cat-dfanalytics.html
+        # @see http://www.elastic.co/guide/en/elasticsearch/reference/8.1/cat-dfanalytics.html
         #
         def ml_data_frame_analytics(arguments = {})
+          arguments = arguments.clone
           headers = arguments.delete(:headers) || {}
 
           body = nil
 
-          arguments = arguments.clone
-
           _id = arguments.delete(:id)
 
           method = Elasticsearch::API::HTTP_GET