From ce8663a0ccbaaaadebe6794efb5adac18aa49e2a Mon Sep 17 00:00:00 2001 From: Earlopain <14981592+Earlopain@users.noreply.github.com> Date: Wed, 10 Apr 2024 17:37:41 +0200 Subject: [PATCH] Remove duplicate `delete_by_query_rethrottle` method definition Closes #230 Signed-off-by: Earlopain <14981592+Earlopain@users.noreply.github.com> --- CHANGELOG.md | 1 + .../api/actions/delete_by_rethrottle.rb | 55 ------------------- 2 files changed, 1 insertion(+), 55 deletions(-) delete mode 100644 lib/opensearch/api/actions/delete_by_rethrottle.rb diff --git a/CHANGELOG.md b/CHANGELOG.md index a7b62f5df..1fa0dc510 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) ### Fixed - Don't emit Ruby warnings when requiring `opensearch-dsl` ([#231](https://github.com/opensearch-project/opensearch-ruby/issues/231)) - Fix release workflow ([#232](https://github.com/opensearch-project/opensearch-ruby/issues/232)) +- Remove duplicated `delete_by_query_rethrottle` to prevent a Ruby warning ([#230](https://github.com/opensearch-project/opensearch-ruby/issues/230)) ### Security ## [3.2.0] diff --git a/lib/opensearch/api/actions/delete_by_rethrottle.rb b/lib/opensearch/api/actions/delete_by_rethrottle.rb deleted file mode 100644 index 1accc69d7..000000000 --- a/lib/opensearch/api/actions/delete_by_rethrottle.rb +++ /dev/null @@ -1,55 +0,0 @@ -# SPDX-License-Identifier: Apache-2.0 -# -# The OpenSearch Contributors require contributions made to -# this file be licensed under the Apache-2.0 license or a -# compatible open source license. -# -# Modifications Copyright OpenSearch Contributors. See -# GitHub history for details. -# -# Licensed to Elasticsearch B.V. under one or more contributor -# license agreements. See the NOTICE file distributed with -# this work for additional information regarding copyright -# ownership. Elasticsearch B.V. licenses this file to you under -# the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. -# You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, -# software distributed under the License is distributed on an -# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY -# KIND, either express or implied. See the License for the -# specific language governing permissions and limitations -# under the License. - -module OpenSearch - module API - module Actions - # The value of requests_per_second can be changed on a running delete by query using the _rethrottle API - # - # @option arguments [String] :task_id The task id to rethrottle (*Required*) - # @option arguments [Number] :requests_per_second The throttle to set on this request in floating sub-requests per second. -1 means set no throttle. - # - # - def delete_by_query_rethrottle(arguments = {}) - raise ArgumentError, "Required argument 'task_id' missing" unless arguments[:task_id] - - method = OpenSearch::API::HTTP_POST - path = "_delete_by_query/#{arguments[:task_id]}/_rethrottle" - params = OpenSearch::API::Utils.__validate_and_extract_params arguments, ParamsRegistry.get(__method__) - body = nil - - perform_request(method, path, params, body).body - end - - # Register this action with its valid params when the module is loaded. - # - # @since 6.2.0 - ParamsRegistry.register(:delete_by_query_rethrottle, [ - :requests_per_second - ].freeze) - end - end -end