search

jsr107 / jsr107spec

JSR107 Cache Specification
Apache License 2.0
413 stars 164 forks source link

Correct Cache.invoke() and Cache.invokeAll() JavaDoc #380

Closed cruftex closed 6 years ago

cruftex commented 7 years ago

From the discussion in https://github.com/jsr107/RI/issues/54:

The sentence:

If an {@link Entry} does not exist for the specified key, an attempt is made to load it (if a loader is configured) or a surrogate {@link Entry}, consisting of the key with a null value is used instead.

needs to be removed from both methods. Reasons:

  1. There should be no effect on the cache at all if the entry processor is doing nothing or is throwing an exception.
  2. In the EntryProcessor JavaDoc it is defined that the loader is invoked when Entry.getValue is called
cruftex commented 6 years ago

@gregrluck Can you please double check an remove the sentence? I don't think its worth a pull request.