Use ReleasableLock within Engine and Translog related classes

[ashking94]

Is your feature request related to a problem? Please describe.
Follow of #11330 (comment)

Describe the solution you'd like
Replace sempahore with ReleasableLock

[macohen]

@ashking94 can you please give a little more background in this issue with why ReleasableLock should replace semaphore? It will help to provide some more context here to categorize and help make it more findable for the community. Thanks!

[ashking94]

@ashking94 can you please give a little more background in this issue with why ReleasableLock should replace semaphore? It will help to provide some more context here to categorize and help make it more findable for the community. Thanks!

There was comment here about this which you can refer. We already use releasable locks at multiple places in our Engine, Translog and other core classes. We have introduced usage of semaphore for the same purpose (as releasable locks) at multiple places. This is mostly a nitpick task - to ensure that we are using similar constructs and standards (i.e. releasable locks instead of semaphores) across these classes.

[ahnyujin]

@ashking94 Is this issue available? I'd like to work on it.

[ashking94]

@ashking94 Is this issue available? I'd like to work on it.

Yes @ahnyujin.