Boris Treukhov created CAMEL-10454:
--------------------------------------

             Summary: Unclear piece in IdempotentConsumer.java
                 Key: CAMEL-10454
                 URL: https://issues.apache.org/jira/browse/CAMEL-10454
             Project: Camel
          Issue Type: Improvement
          Components: camel-core
            Reporter: Boris Treukhov
            Priority: Minor


Current implementation of IdempotentConsumer.java has very strange check on 
whether IdempotentRepository contains the key - if it's 
ExchangeIdempotentRepository then the result of "contains()" check is not 
inverted(i.e. there is no NOT operator), while for other types of 
IdempotentRepository it IS inverted with "!".

 // check if we already have the key
                if (idempotentRepository instanceof 
ExchangeIdempotentRepository) {
                    newKey = ((ExchangeIdempotentRepository<String>) 
idempotentRepository).contains(exchange, messageId);
                } else {
                    newKey = !idempotentRepository.contains(messageId);
                }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to