[HDDS-6990] AuthorizerLockImpl.java#tryWriteLockInOMRequest: move sanity arg check to after tryWriteLockThrowOnTimeout();

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: Ozone Manager
• Labels: pull-request-available
• Assignee: George Huang
• Reporter: George Huang
• Created: 11/Jul/22 05:08
• Updated: 16/Feb/23 05:26
• Resolved: 13/Jul/22 20:37

Description

IllegalArgumentException from Preconditions check in tryWriteLockInOMRequest when another tenant write request is in-progress

The IllegalArgumentException looks scary and misleading. We're now moving the sanity arg check to after the call of 

long stamp = tryWriteLockThrowOnTimeout();

Issue Links

• relates to: HDDS-7001 OMTenantCreateRequest#validateAndUpdateCache might not release lock when exception is thrown
• links to: GitHub Pull Request #3588