[COLLECTIONS-475] Wrong timeout handling in expiration policies of PassiveExpiringMap

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0-alpha1
• Fix Version/s: 4.0
• Assignee: Unassigned
• Reporter: Thomas Neidhart
• Created: 02/Jul/13 21:04
• Updated: 09/Nov/14 14:31
• Resolved: 14/Jul/13 19:09

Description

The timeunit parameter in the ctor of ConstantTimeToLiveExpirationPolicy is not used.

Additionally, the validateAndConvertToMillis method is wrong, it should be changed like this:

    private static long validateAndConvertToMillis(final long timeToLive,
                                                   final TimeUnit timeUnit) {
  if (timeUnit == null) {
    throw new IllegalArgumentException("Time unit must not be null");
  }
  return TimeUnit.MILLISECONDS.convert(timeToLive, timeUnit);
}