Синхронизируйте метод с разными потоками Sidekiq и подождите.
-
21-12-2019 - |
Вопрос
Проблема:У меня есть несколько потоков Sidekiq и функция, которую можно вызывать только один раз из любого потока.
Причина:Мы запрашиваем API AdWords, чтобы получить некоторые данные.Они довольно строгие, когда дело доходит до пределов ставок.Только один из потоков может одновременно вызывать функцию для получения данных.
Теперь немного кода:
# Public: Get estimates for a set of keywords. If there is an error, retry
# several times. If not successful, raise an error
#
# keywords: The keyword objects to get estimates for.
# save: Boolean to indicate whether the keyword objects should be saved to
# the database
#
def repeatedly_try_get_estimates(keywords: [], save: true, sleep_delay: 150)
return keywords if keywords.empty?
func = -> { get_estimates(keywords, !save) }
retry_operation(function: func, max_tries: 15, sleep_delay: sleep_delay)
end
- Как видите, прямо сейчас у меня есть огромный
sleep_delay
обойти проблему. - Код вызывает
retry_operation
функционировать сget_estimates
функция как параметр.Затем он повторит попыткуget_estimates
Функция несколько раз, пока не появится исключение API.
А retry_function
:
# Private: Retry a function X times and wait X seconds. If it does not work X times,
# raise an error. If successful return the functions results.
#
# - max_tries: The maximum tries to repeat the function
# - sleep_delay: The seconds to wait between each iteration.
# - function: The lambda function to call each iteration
#
def retry_operation(max_tries: 5, sleep_delay: 30, function: nil, current_try: 0, result: nil)
# Can't call, no function
if function.nil?
return
end
# Abort, tried too frequently.
if current_try > max_tries
raise "Failed function too often"
end
# Check if there is an exception
exception = true
begin
result = function.call
exception = false
rescue => e
Rails.logger.info "Received error when repeatedly calling function #{e.message.to_s}"
end
if exception
sleep sleep_delay if sleep_delay > 0
retry_operation(max_tries: max_tries, sleep_delay: sleep_delay, function: function, current_try: current_try + 1)
else
result
end
end
А get_estimates_function
здесь: https://gist.github.com/a14868d939ef0e34ef9f.Это слишком долго, на всякий случай.
Думаю, мне нужно сделать следующее:
- Откорректируйте код в
repeatedly_try_get_estimates
функция. - Используйте мьютекс в классе.
- Спасите исключение, если мьютекс используется.
- Только если мьютекс свободен, запустите
rety_operation
, еще поспи немного
Спасибо за вашу помощь :)
Решение
Вот и все, заработало:
# Public: Get estimates for a set of keywords. If there is an error, retry
# several times. If not successful, raise an error
#
# keywords: The keyword objects to get estimates for.
# save: Boolean to indicate whether the keyword objects should be saved to
# the database
#
def repeatedly_try_get_estimates(keywords: [], save: true, sleep_delay: 40)
return keywords if keywords.empty?
func = -> { get_estimates(keywords, save_keywords: true) }
exception = nil
result = nil
initial_sleep = 0
estimates_mutex.synchronize do
since_last_request = Time.now.to_i - last_adwords_api_request
if since_last_request <= 30
Rails.logger.info "AdWords: Last request was only few seconds ago - sleeping #{since_last_request}."
initial_sleep = since_last_request
end
begin
result = retry_operation(function: func, max_tries: 15, sleep_delay: sleep_delay, initial_sleep: initial_sleep)
rescue => e
exception = e
end
@@last_adwords_api_request = Time.now.to_i
end
if exception
raise exception
end
result
end
Не связан с StackOverflow