[Storage] [DataMovement] Remove unnecessary synchronization primitives (e.g. interlocked, semaphores)

[amnguye]

Library name and version

Azure.Storage.DataMovement; Azure.Storage.DataMovement.Blobs

Query/Question

In the CommitChunkHandler, DownloadChunkHandler there are some unnecessary synchronization primitives (e.g. interlocked, semaphoreslim) which may not need and may be overkill to the purpose of preventing synchronization problems.

Let's analyze, remove and test to see which ones we need, to improve performance.

EDIT: Other places to consider:

• Transfer status locks: Read, Write, Update from Job
• Job Part Status Update

Environment

No response