Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1901#discussion_r121038362
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-distributed-cache-client-service-api/src/main/java/org/apache/nifi/distributed/cache/client/AtomicDistributedMapCacheClient.java
---
@@ -71,6 +73,23 @@
* @return true only if the key is replaced.
* @throws IOException if unable to communicate with the remote
instance
*/
+ @Deprecated
<K, V> boolean replace(K key, V value, Serializer<K> keySerializer,
Serializer<V> valueSerializer, long revision) throws IOException;
--- End diff --
@bbende Yeah, actually, I'm writing Proof of Concept code and it will be
ready in next few hours hopefully. I will share it on Github later so that you
can take a look, if it looks ok, then lets update the
AtomicDistributedMapCacheClient in another JIRA so that both of your Redis
cache and my Couchbase cache can use it.
I found another benefit of this approach,, we may not have to add new
protocol version. I'm testing the idea now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
