Как реализовать экспоненциальную задержку / сон для try & catch - PullRequest
Новая
       31

Как реализовать экспоненциальную задержку / сон для try & catch

0 голосов
/ 06 февраля 2019

Я пытаюсь написать другой декоратор, как RetriableProcessorDecorator ниже (как отдельный класс), чтобы он делал экспоненциальную задержку при повторных попытках.Например, если сообщение не может быть обработано, мы ждем 1 секунду (должна быть настраиваема), затем 2 с, затем 4 с, затем 8 с, затем 16 с и т. Д. Я бы хотел использовать поток вместо ожидания занятости, поскольку это дешевле,Я написал новый класс RetriableProcessorExponentialDecorator, который делает это, но я не уверен, что это правильный подход.

RetriableProcessorDecorator.java: 

@Slf4j
@Setter
@RequiredArgsConstructor
@AllArgsConstructor(access = AccessLevel.PACKAGE)
public class RetriableProcessorDecorator implements.   
AbsMessageProcessorDecorator {
private final AbsMessageProcessor messageProcessor;
@Autowired
private AbsMessageActiveMQConfiguration configuration;

@Override
public void onMessage(AbsMessage message) throws Exception {
    int executionCounter = 0;
    final int maxRetries = this.configuration.getExceptionRetry() + 1;
    do {
        executionCounter++;
        try {
            this.messageProcessor.onMessage(message);
        } catch (RetriableException e) {
            log.info("Failed to process message. Retry #{}", executionCounter);
        } catch (Exception e) {
            // We don't retry on this, only RetriableException.
            throw e;
        }
    } while (executionCounter < maxRetries);
}
}

RetriableProcessorExponentialDecorator.java (новый класс я реализую): 

public class RetriableProcessorExponentialDecorator implements AbsMessageProcessorDecorator {
  private final AbsMessageProcessor messageProcessor;
  @Autowired
  private AbsMessageActiveMQConfiguration configuration;

  @Override
  public void onMessage(AbsMessage message) throws Exception {
    int executionCounter = 0;
    int delayCounter = 1000;
    final int maxRetries = this.configuration.getExceptionRetry() + 1;
    do {
      executionCounter++;
      try {
        this.messageProcessor.onMessage(message);
      } catch (RetriableException e) {
        log.info("Failed to process message. Retry #{}", executionCounter);
        Thread.sleep(delayCounter);
        delayCounter = delayCounter * 2;
      } catch (Exception e) {
        // We don't retry on this, only RetriableException.
        throw e;
      }
    } while (executionCounter < maxRetries && delayCounter < Long.MAX_VALUE);
  }
}

1 Ответ

0 голосов
/ 06 февраля 2019

В целом, я думаю, что ваш подход хорош.

Однако, если вы хотите сделать его настраиваемым (то есть более полезным), вы можете сделать что-то вроде: 

double multiplier = 2.0; // make this configurable
...
delayCounter = (long) (delayCounter * (Math.pow(multiplier, executionCounter)));
Thread.sleep(delayCounter);

Вы могли бытакже добавьте способ настройки максимальной задержки, которая может быть полезна для некоторых случаев использования, например: 

long maxDelay = 300000; // 5 minutes
...
if (delayCounter > maxDelay) {
   delayCounter = maxDelay;
}
Thread.sleep(delayCounter);
...