[HBASE-15664] Use Long.MAX_VALUE instead of HConstants.FOREVER in CompactionPolicy

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.0, 0.98.19, 1.4.0, 2.0.0
• Fix Version/s: 1.3.0, 0.98.19, 2.0.0
• Component/s: Compaction
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 17/Apr/16 11:19
• Updated: 09/Nov/17 00:42
• Resolved: 18/Apr/16 19:15

Description

The TTL per CF is in seconds, we will convert it to milliseconds when construct HStore. And if it is HConstants.FOREVER, we will set it to Long.MAX_VALUE.

HStore.java

  public static long determineTTLFromFamily(final HColumnDescriptor family) {
    // HCD.getTimeToLive returns ttl in seconds.  Convert to milliseconds.
    long ttl = family.getTimeToLive();
    if (ttl == HConstants.FOREVER) {
      // Default is unlimited ttl.
      ttl = Long.MAX_VALUE;
    } else if (ttl == -1) {
      ttl = Long.MAX_VALUE;
    } else {
      // Second -> ms adjust for user data
      ttl *= 1000;
    }
    return ttl;
  }