[HBASE-21511] Remove in progress snapshot check in SnapshotFileCache#getUnreferencedFiles

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 1.3.3, 2.2.0, 1.4.9, 2.1.2
• Component/s: snapshots
• Hadoop Flags: Reviewed
• Assignee: Ted Yu
• Reporter: Ted Yu
• Created: 24/Nov/18 11:15
• Updated: 26/May/20 20:47
• Resolved: 26/Nov/18 05:50

Description

During review of HBASE-21387, Apache9 mentioned that the check for in progress snapshots in SnapshotFileCache#getUnreferencedFiles is no longer needed now that snapshot hfile cleaner and taking snapshot are mutually exclusive.

This issue is to address the review comment by removing the check for in progress snapshots.

Issue Links

• relates to: HBASE-21387 Race condition surrounding in progress snapshot handling in snapshot cache leads to loss of snapshot files
• relates to: HBASE-23202 ExportSnapshot (import) will fail if copying files to root directory takes longer than cleaner TTL
• relates to: HBASE-24442 Test coverage for HBASE-23202 - ExportSnapshot (import) will fail if copying files to root directory takes longer than cleaner TTL