[
https://issues.apache.org/jira/browse/CAMEL-12041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16275944#comment-16275944
]
ASF GitHub Bot commented on CAMEL-12041:
----------------------------------------
GitHub user drekbour opened a pull request:
https://github.com/apache/camel/pull/2127
CAMEL-12041 Introduce ConcurrentMapIdempotentRepository and
MapIdempotentRepository
+ Introduce AbstractIdempotentRepositoryTest
Rebased, tested and checkstyle'd.
The new tests duplicate and obselete a great number of the existing
Idempotent* and FileIdempotent* tests but I have NOT deleted the pre-existing
ones. I can update this PR with those deletions if requested.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/drekbour/camel CAMEL-12041
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/2127.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2127
----
commit 4355dc2073ed1103cf80269d013944de2aded4d8
Author: Marc Carter <[email protected]>
Date: 2017-12-02T10:02:22Z
CAMEL-12041 Introduce ConcurrentMapIdempotentRepository and
MapIdempotentRepository
Introduce AbstractIdempotentRepositoryTest
----
> Introduce ConcurrentMapIdempotentRepository and MapIdempotentRepository
> -----------------------------------------------------------------------
>
> Key: CAMEL-12041
> URL: https://issues.apache.org/jira/browse/CAMEL-12041
> Project: Camel
> Issue Type: Improvement
> Components: camel-core
> Affects Versions: 2.20.1
> Reporter: Marc Carter
> Priority: Trivial
>
> In order to more generically plug in third-party data fabrics, Camel should
> directly support the ConcurrentMap interface.
> 1 Add an explicit ConcurrentMapIdempotentRepository (which does away with
> synchronized blocks around the backing map)
> 2 Factor our MapIdempotentRepository from MemoryIdempotentRepository (being
> based on a Map _interface_ doesn't make it "memory" - only the default
> LRUCache impls are definitively "memory")
> 3 Make some sense of the tests around this EIP
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
