You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by "Michael Kobit (JIRA)" <ji...@apache.org> on 2015/12/18 16:54:46 UTC

[jira] [Created] (NIFI-1305) Clarify Jacadoc on DistributedMapCacheClient.getAndPutIfAbsent

Michael Kobit created NIFI-1305:
-----------------------------------

             Summary: Clarify Jacadoc on DistributedMapCacheClient.getAndPutIfAbsent
                 Key: NIFI-1305
                 URL: https://issues.apache.org/jira/browse/NIFI-1305
             Project: Apache NiFi
          Issue Type: Improvement
          Components: Documentation & Website
    Affects Versions: 0.4.0
            Reporter: Michael Kobit
            Priority: Trivial


The Javadoc for this method on DistributedMapCacheClient (https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-services/nifi-distributed-cache-client-service-api/src/main/java/org/apache/nifi/distributed/cache/client/DistributedMapCacheClient.java#L53-L71) is unclear about what happens if the key does not exist in the cache.

It looks like from the implementation of DistributedMapCacheClientService that it returns the value if it present, otherwise returns the computed value. The Javadoc doesn't state what is returned happens if an item does not exist.

I spent some time trying to figure out what makes sense from a client's perspective, and I had some trouble trying to understand what should happen. It seems obvious to me now that the cache should return either the existing value or the newly computed value, but it took some time to get there. 

I think the documentation should be clearer about what is returned in the case where the item does not exist



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