[ACCUMULO-2371] cachedDataIndex.close() should be enclosed in finally block in BCFile#Reader()

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.6.0
• Assignee: Eric C. Newton
• Reporter: Ted Yu
• Created: 14/Feb/14 19:27
• Updated: 20/Feb/14 15:11
• Resolved: 20/Feb/14 15:11

Description

Here is related code:

        try {
          dataIndex = new DataIndex(cachedDataIndex);
        } catch (IOException e) {
          LOG.error("Got IOException when trying to create DataIndex block");
          throw e;
        }
        cachedDataIndex.close();

If there is IOE, cachedDataIndex.close() would be skipped.