smallrye-mutiny
smallrye-mutiny copied to clipboard
Rename `UniMemoize#atLeast`
Discussed in https://github.com/smallrye/smallrye-mutiny/discussions/742
Originally posted by michalszynkiewicz November 5, 2021
For me, atLeast
suggests that the value will be memoized/cached for the specified amount of time or longer (I'd guess might be stored in a weak reference after the period of time).
The discussion on a Stork PR says that the method is actually specifying time to live for the cached value.
Could the method be renamed to e.g. forDuration
?
CC @jponge
/cc @michalszynkiewicz
Is this issue still open?
Yes, as we didn't find a good alternative.
How about something like forFixedDuration
? IMHO this would clarify that the memoization is only in effect for the specified amount of time.
Ah, Nice. WDYT @jponge
Yes that sounds good
Great, I can provide a PR then :)