[HDFS-11246] FSNameSystem#logAuditEvent should be called outside the read or write locks

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.7.3
• Fix Version/s: 3.3.0
• Assignee: Xiaoqiao He
• Reporter: Kuhu Shukla
• Created: 14/Dec/16 22:05
• Updated: 02/Oct/19 17:13
• Resolved: 29/Aug/19 17:13

Description

    readLock();
    boolean success = true;
    ContentSummary cs;
    try {
      checkOperation(OperationCategory.READ);
      cs = FSDirStatAndListingOp.getContentSummary(dir, src);
    } catch (AccessControlException ace) {
      success = false;
      logAuditEvent(success, operationName, src);
      throw ace;
    } finally {
      readUnlock(operationName);
    }

It would be nice to have audit logging outside the lock esp. in scenarios where applications hammer a given operation several times.